Skip to content
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

Update project configurator designs #4948

Closed
gtsiolis opened this issue Jul 26, 2021 · 5 comments · Fixed by #5236
Closed

Update project configurator designs #4948

gtsiolis opened this issue Jul 26, 2021 · 5 comments · Fixed by #5236
Assignees

Comments

@gtsiolis
Copy link
Contributor

gtsiolis commented Jul 26, 2021

Following the updates in project #4631 which added a cut through of the project configurator, the project configurator designs need to be updated.

Designs

Adding below the first step of the project configurator.

Authorize Provider
Pending
Repository List Repository Hover Repository Taken
Default Hover Taken
Select Team Create Team
Default Create
Configuration Existing in Git
Configure
Add Configuration Pending Prebuild Failed Prebuild Running Prebuild Ready prebuild
Configure-4 Configure Configure-1 Configure-2 Configure-3
Project Configuration
Prebuilds

See design specs.

@jankeromnes
Copy link
Contributor

Happy to take this on next week! Also, happy to collaborate on this (e.g. by reviewing small PRs in case you want to fix small parts of this 🏓)

@gtsiolis
Copy link
Contributor Author

gtsiolis commented Aug 5, 2021

LET'S DO THIS ! 🏀

@gtsiolis
Copy link
Contributor Author

gtsiolis commented Aug 10, 2021

Pasting below the new specs for the project configuration page. Cc @jankeromnes @AlexTugarev @svenefftinge

Detecting Detected
Detecting Detected
Existing Failed Pending Success
Existing Failed Pending Success

@jankeromnes
Copy link
Contributor

jankeromnes commented Aug 10, 2021

Wow! These specs are extremely good ✨ 😍 🎯

Many thanks! 🙏 Will start implementing them in my next PR 🚀

@gtsiolis
Copy link
Contributor Author

Thanks, @jankeromnes! Let me know if anything looks unclear. 🤓

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants