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

Updated Build & Deploy Message #298

Closed
wants to merge 3 commits into from

Conversation

nicoleiftekhar
Copy link
Contributor

Updated the build and deploy message to display that the request may take a few minutes to process. The message added includes...

"Please note that it usually takes a couple minutes for the instance to be deployed and the link above may temporarily display an error message."

@nicoleiftekhar nicoleiftekhar requested a review from ryanmohta May 12, 2023 22:11
@nicoleiftekhar nicoleiftekhar removed the request for review from ryanmohta May 12, 2023 22:12
@github-actions
Copy link

Deployed staging instance to https://staging-298.peterportal.org Please note that it usually takes a couple minutes for the instance to be deployed and the link above may temporarily display an error message.

Copy link
Member

@ethanwong16 ethanwong16 left a comment

Choose a reason for hiding this comment

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

LGTM! Left a minor suggestion

@@ -109,4 +109,4 @@ jobs:
with:
header: staging url
recreate: true
message: Deployed staging instance to https://staging-${{ github.event.pull_request.number }}.peterportal.org
message: Deployed staging instance to https://staging-${{ github.event.pull_request.number }}.peterportal.org Please note that it usually takes a couple minutes for the instance to be deployed and the link above may temporarily display an error message.
Copy link
Member

Choose a reason for hiding this comment

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

Great work! A minor suggestion, were you able to look into potentially adding a newline between the previous message and the new one you added?

@js0mmer
Copy link
Member

js0mmer commented Nov 1, 2023

Closing for now since it no longer seems to be necessary with our current infrastructure (deploying with SST). Instance seems to be up as soon as the link is commented now.

@js0mmer js0mmer closed this Nov 1, 2023
@js0mmer js0mmer deleted the nicole-update-deploy-msg branch November 21, 2023 08:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants