-
Notifications
You must be signed in to change notification settings - Fork 14
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
Conversation
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. |
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.
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. |
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.
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?
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. |
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."