clockworklabs / clockworklabs/SpacetimeDB
Implement FilterableValue for Option<T>
Nobody has claimed this yet.
- Dominant language
- Rust
- Stars
- 25.2k
- Forks
- 1.1k
- Avg merge
- 2d 7h
- Merged PRs (30d)
- 46
Description
This issue tracks reimplementing the work from stale PR #4013, which has been closed because it is too far out of date to merge directly.
Original pull request
- PR #4013: Implemented
FilterableValueforOption<FilterableValue> - Original author: @egormanga
What the original PR was trying to do
Description of Changes * Resolves 3992 API and ABI breaking changes None. Expected complexity level and risk 1. Testing - [x] Manual testing - [ ] Unit tests
Closure context
- I'm closing this one to be reimplemented as this is not a complete implementation.
Reimplementation notes
- Reimplement this work on top of current
masterin a new PR. - Keep the original PR linked as historical context and as a source of useful implementation ideas where still relevant.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reviewing stale PR #4013 and then locate the current FilterableValue trait and Option implementation on master. Compare the historical changes with the current API and ABI, and consider how the intended behavior can be verified; done means a complete current implementation rather than a direct reuse of the outdated PR.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- backend-api-design
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100