Skip to content

[public-api] Create token UI #14899

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.

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

Merged
merged 1 commit into from
Nov 24, 2022
Merged

[public-api] Create token UI #14899

merged 1 commit into from
Nov 24, 2022

Conversation

jeanp413
Copy link
Member

@jeanp413 jeanp413 commented Nov 23, 2022

Related Issue(s)

Fixes #14615

Release Notes

NONE

Documentation

Werft options:

  • /werft with-local-preview
    If enabled this will build install/preview
  • /werft with-preview
  • /werft with-large-vm
  • /werft with-integration-tests=all
    Valid options are all, workspace, webapp, ide, jetbrains, vscode, ssh

@jeanp413 jeanp413 requested a review from a team November 23, 2022 17:24
@werft-gitpod-dev-com
Copy link

started the job as gitpod-build-jp-public-api-create-ui.1 because the annotations in the pull request description changed
(with .werft/ from main)

@jeanp413 jeanp413 marked this pull request as draft November 23, 2022 17:24
@github-actions github-actions bot added the team: webapp Issue belongs to the WebApp team label Nov 23, 2022
@jeanp413 jeanp413 marked this pull request as ready for review November 24, 2022 11:40
@easyCZ
Copy link
Member

easyCZ commented Nov 24, 2022

This is awesome!

Attaching screenshots just for completness:
Screenshot 2022-11-24 at 12 58 47
Screenshot 2022-11-24 at 12 58 28
Screenshot 2022-11-24 at 12 58 23

@easyCZ
Copy link
Member

easyCZ commented Nov 24, 2022

/hold for scopes change

Copy link
Member

@easyCZ easyCZ left a comment

Choose a reason for hiding this comment

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

LGTM.

I'm not focusing on the UI/UX polish too much at this point as making it functional first is the priority. We can do another pass on UX adjustments later.

Feel free to remove the hold once the scopes are updated

@gtsiolis
Copy link
Contributor

I'm not focusing on the UI/UX polish too much at this point as making it functional first is the priority. We can do another pass on UX adjustments later.

ok

@easyCZ
Copy link
Member

easyCZ commented Nov 24, 2022

I've applied the suggested fix so we can land this. It remains behind a feature flag so future iterations to fix up are easy to do. Will unhold once the build passes.

@mustard-mh mustard-mh force-pushed the jp/public-api-create-ui branch from 0fba79b to 0216592 Compare November 24, 2022 12:47
@mustard-mh
Copy link
Contributor

mustard-mh commented Nov 24, 2022

Rebased to main, going to do some 🧹

/hold

@mustard-mh mustard-mh force-pushed the jp/public-api-create-ui branch from 0216592 to f64875a Compare November 24, 2022 13:15
Co-authored-by: Pudong <tianshi8650@gmail.com>
Co-authored-by: Huiwen <mhqnwt@gmail.com>
Co-authored-by: Milan Pavlik <pavlik.mil@gmail.com>
@mustard-mh mustard-mh force-pushed the jp/public-api-create-ui branch from f64875a to d0539a6 Compare November 24, 2022 13:22
Copy link
Contributor

@mustard-mh mustard-mh left a comment

Choose a reason for hiding this comment

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

LGTM

/unhold

@mustard-mh
Copy link
Contributor

/werft run

@mustard-mh
Copy link
Contributor

mustard-mh commented Nov 24, 2022

/werft run recreate-preview=true

👍 started the job as gitpod-build-jp-public-api-create-ui.8
(with .werft/ from main)

@roboquat roboquat merged commit 4a4d66d into main Nov 24, 2022
@roboquat roboquat deleted the jp/public-api-create-ui branch November 24, 2022 13:59
@roboquat roboquat added deployed: webapp Meta team change is running in production deployed Change is completely running in production labels Nov 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deployed: webapp Meta team change is running in production deployed Change is completely running in production release-note-none size/XL team: webapp Issue belongs to the WebApp team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement token creation page/modal
5 participants