-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
ci: Add deployment preview to pull requests via Uffizzi integration #2364
ci: Add deployment preview to pull requests via Uffizzi integration #2364
Conversation
Thanks for opening this pull request!
|
The integration looks much more lightweight now, thanks for the new PR. What's the status of this PR and what would we need to do to add this preview to all PRs of the Parse Dashboard repo? I think we could give this a try. |
that's great to hear — I'm glad we could align it to meet Parse's requirements :)
And you'll be good to go! Also, this is how the Uffizzi CI works for now.
Here are our docs on how to use Uffizzi CI if you want to read more about this. |
I assume we need to write our own bot to add the preview URL as a comment to the PR thread? Or is that a feature that is already supported?
I tried that, but got an error: The repository belongs to an org, but the org account |
Hi @mtrezza I'm co-founder at Uffizzi. Happy to help you out.
No you don't need to write your own bot, Uffizzi CI will post the comment for you. You will grant permission to the Uffizzi GitHub App to write to PR issues, as described here. It looks like |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, it found the repository; I'll merge this to continue the configuration in the console.
Continuing discussion in issue #2374.
🎉 This change has been released in version 5.1.0-alpha.7 |
🎉 This change has been released in version 5.1.0-beta.2 |
🎉 This change has been released in version 5.1.0 |
* source: (41 commits) chore(release): 5.1.0 [skip ci] chore(release): 5.1.0-beta.2 [skip ci] ci: Skip unnecessary builds (parse-community#2389) refactor: Upgrade various dependencies (parse-community#2388) chore(release): 5.1.0-alpha.10 [skip ci] fix: Data browser dialog "No data to display" may be outside of visible area in Safari browser (parse-community#2387) chore(release): 5.1.0-alpha.9 [skip ci] fix: Screen goes blank when trying to add column of type `Object` or `GeoPoint` (parse-community#2384) refactor: Bump http-cache-semantics from 4.1.0 to 4.1.1 (parse-community#2381) chore(release): 5.1.0-alpha.8 [skip ci] fix: Internal error message on login with missing credential (parse-community#2370) refactor: Remove warnings in Docker build (parse-community#2350) ci: Restyle preview html (parse-community#2377) chore(release): 5.1.0-alpha.7 [skip ci] fix: Dashboard may display blank page when selecting an app after login (parse-community#2375) ci: Add deployment preview to pull requests via Uffizzi integration (parse-community#2364) chore(release): 5.1.0-alpha.6 [skip ci] fix: Navigation to page fails if user re-login is required (parse-community#2369) refactor: Bump ua-parser-js from 0.7.28 to 0.7.33 (parse-community#2372) chore(release): 5.1.0-alpha.5 [skip ci] ... # Conflicts: # package-lock.json
New Pull Request Checklist
Approach
This PR is opened as an update to PR #2352. This PR is moving
Uffizzi build
andUffizzi preview
workloads from GitHub actions to Uffizzi CI.Uffizzi CI will handle building the containers and provisioning preview environments for PRs. Once the preview is up and ready, a comment will be posted on the PR on GitHub. This requires lesser configuration than GHA integration.
Here is a guide on the Uffizzi CI setup and how it works.
Post merging this PR into Parse's
default
branch, you'll have to follow these steps to configure Uffizzi CI to provision previews for PRs on Parse-dashboard.Here is a test PR which was opened in my fork of Parse-dashboard. The building and provisioning were done outside of GHA and in Uffizzi CI. Post completion, this comment was posted on the PR. This is the preview environment. To log into Parse-dashboard for this test PR, use the following credentials:
cc @mtrezza
TODOs before merging