oxidecomputer / oxidecomputer/console
Convert QueryParamTabs to route tabs
Open
Nobody has claimed this yet.
- Dominant language
- TypeScript
- Stars
- 228
- Forks
- 22
- Avg merge
- 19h 42m
- Merged PRs (30d)
- 32
Description
Sometimes for tabs we use params:
/projects/mock-project/vpcs/mock-vpc?tab=firewall-rules
and sometimes pages:
/projects/mock-project/instances/db1/storage
We should either land on a single approach or document why we use which approach in which context.
Contributor guide
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 with QueryParamTabs and compare the query-parameter example with the route-based instance path described in the issue. Done means the project has one consistent tab approach, or documents clearly which contexts use each approach.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- frontend
- Issue type
- Refactor
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100