-
-
Notifications
You must be signed in to change notification settings - Fork 218
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
[B5] Exports: create new export #35170
Conversation
The old one goes to a redirection page, which is a worse experience.
Tests fail because I'm holding off on running I'm not cleaning up the old B3 templates & updating migration status for the same reason, I'll do that in a later PR. |
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.
Commits up to aefedd4 look good.
I assume you're planning to add an additional commit after running build_bootstrap5_diffs
just before this is ready to merge. Is that correct?
@millerdev yes |
QA passed and #35161 is merged. I added the B5 diffs, this is now ready to go. |
Product Description
Joint effort with @AmitPhulera
User-facing changes are pretty minor.
Before
After
Safety Assurance
Safety story
UI-level changes. Requesting QA.
Automated test coverage
I don't think so
QA Plan
https://dimagi.atlassian.net/browse/QA-7132 - joint QA with #35161
Rollback instructions
Labels & Review