influxdata / influxdata/ui

Migrate initial global header fetch away from accounts

Open
#6,011 0 comments 0 reactions 0 assignees View on GitHub
kind/bug kind/tech-debt team/dumplings team/ui
Dominant language
TypeScript
Stars
117
Forks
51
Avg merge
2d 15h
Merged PRs (30d)
4

Description

After #5829 , we will no longer be using React's Context API for quartz /accounts data. Once that is the case, global header should be modified as follows:

- Initially store only the user's current account/org based on data from /identity. Do not store full list of accounts until menu is expanded.
- Change conditional for whether to render switch accounts button from checking for the _length_ of the /accounts array to checking the _value_ of the account/org count properties from the user object.
- Fetch /accounts _only_ once global header is expanded.
- Discuss with design appropriate behavior when header is in a 'loading' state of fetching (or failing to fetch) from /accounts.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start with the global header and the #5829 Context API removal, then trace its current /identity and /accounts data flow. Done means the header initially uses only the current account/org, fetches /accounts on expansion, checks count properties for the switch button, and has an agreed loading or failure behavior with design.

Written by the indexing model from the issue text.

Assessment

Tech stack
react, typescript
Domain
frontend
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.