-
Notifications
You must be signed in to change notification settings - Fork 5
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
QA for 4/17/2024 production deploy #1429
Comments
I think we've already captured this, but noting here again that when you go to create a new template and copy from an existing template, you don't see a list of any of the existing templates and can only search by name, which means you have to remember which templates exist. |
@ccostino Everything on the QA list worked for me except the items listed below. Send a message from copied template
![]()
Send a message from existing template
![]() |
Awesome, thanks for capturing these @heyitsmebev! We'll follow up to see if we're dealing with known issues or need to create new ones, and if any of these are actual show-stoppers. As for the sign-in again form there, that's a known issue that @em-herrick has already flagged that still needs to be taken care of - it's on our list of TODOs above that @jonathanbobel is working on! 🙂 |
Ah yes, this is what I was seeing yesterday too in the set up your profile form, thanks @em-herrick! Update: now it's own issue |
@ccostino The allowance updates to ![]() I tested locally in dev and the values update. Possibly a staging issue? |
🤔 I'm not sure, @A-Shumway42, so once the service was switched to live the updates to the counts stopped working? Is this an existing service that was creating a while back, or a new one you just set up for testing? |
While investigating #1427 I encountered two broken links: Both lead to a "Page not found" at these two URLs:
The double slash ( As a result of this I can't say for sure if user story 1427 is complete or not |
Added an issue for the "you need to sign back in" template - the Login.gov button is missing. |
@ccostino this was a new service in staging. I logged back into staging and it appears it is working now. I'm not sure what the delay was on the count. |
Oh strange, @A-Shumway42! Two things come to mind that we could look into further to see:
|
When investigating issue #1270 I found that there is some text in the google doc draft of email language which does not appear in the email:
Unsure if this was actually supposed to be included, or if it was old and discarded text |
@ccostino Hard to verify now. I'm not receiving messages to my phone and I've sent a few more in staging. The dashboard still has a count of 2 in the count at the bottom and the top. I also can't access logs in staging because my cf cli can't login...oof. |
@A-Shumway42 okay - I noticed some delays with messages going out yesterday myself - you might be able to see some activity (or not!) in AWS in the staging account, but I think we're okay for now and we can follow up on this more if need be. @heyitsmebev, thanks for all of your reports there! It looks like the discrepancies with the templates are more so with the QA checklist and not the site itself, so we'll have to update that issue template. 🙂 As for the sign in again page, that's tracked here and nearly wrapped up! @jskinne3, thanks for reporting these! Sure enough those are bugs caused by the current information architecture of all things. Those two links are tied to a service and only work if you've clicked into a service first - they are settings page for a service, not global/universal for just your user account. So thank you for flagging, @em-herrick and I confirmed this just a bit ago! As for the template, thanks for flagging that too; it looks like it didn't get fully updated for some reason, and highlights that the template updating process is still a bit wonky. @em-herrick and I will write up separate issues to track the things that we do need to follow up on from all of this. Thank you all! |
Our production release has gone out and new issues have been created. Thanks, everyone! |
Log In
Create a template and send
Send a message from copied template
Send a message from existing template
prepare to send
buttonDownload the csv delivery report
Invite a team member
Delete a team member
Sign Out
Specific Issues to review
We have a handful of issues that are ready to be reviewed or could use verification whether or not they're fixed or need more follow up:
TODO
We're waiting on a couple of things to still be completed and merged to staging:
The text was updated successfully, but these errors were encountered: