Skip to content

442 UI to acquire api key #450

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
longshuicy merged 8 commits into from
Apr 21, 2023
Merged

442 UI to acquire api key #450

longshuicy merged 8 commits into from
Apr 21, 2023

Conversation

@longshuicy
Copy link
Member

@longshuicy longshuicy commented Apr 19, 2023

Things covered in this PR:

  • convert the Logout link on the topbar to user menu with avatar
  • Menu item to generate apikey
  • Dialog to generate and copy the key to clipboard
  • Close the Dialog will reset the key so it won't show up again

image

image

image

@longshuicy longshuicy linked an issue Apr 19, 2023 that may be closed by this pull request
UI to acquire api key #442
Closed
max-zilla
max-zilla approved these changes Apr 21, 2023
View reviewed changes
Copy link
Contributor

@max-zilla max-zilla left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

everything works well

@longshuicy longshuicy merged commit b2af538 into main Apr 21, 2023
arunapa pushed a commit that referenced this pull request Apr 27, 2023
@longshuicy @arunapa
442 UI to acquire api key (#450)
6a71cdf 
* add apikey to redux

* API key modal

* topbar icons

* add connection to popup

* temp save

* styling and add copy to clipboard

* add copy to clipboard capability

* reset after creation
@max-zilla max-zilla deleted the 442-ui-to-acquire-api-key branch July 20, 2023 19:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@max-zilla max-zilla max-zilla approved these changes

@lmarini lmarini Awaiting requested review from lmarini

@arunapa arunapa Awaiting requested review from arunapa

@tcnichol tcnichol Awaiting requested review from tcnichol

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

UI to acquire api key

3 participants

@longshuicy @max-zilla