Skip to content

feat: Update response options banner to September 30 (M2-11051) - #2254

Merged
divbzero merged 2 commits into
developfrom
M2-11051-response-options-layout-banner-0930
Aug 20, 2026
Merged

feat: Update response options banner to September 30 (M2-11051)#2254
divbzero merged 2 commits into
developfrom
M2-11051-response-options-layout-banner-0930

Conversation

@divbzero

@divbzero divbzero commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

📝 Description

🔗 Jira Ticket M2-11051

Changes include:

  • Update date for changing response options layout September 16 → September 30
  • NFC: Update heading level on PR checklist

✏️ Notes

This is a follow-up to #2252. As discussed, this allows more advance notice for admins.

✅ Checklist

Functionality

  • The feature behaves correctly in practice and fulfills the intended business purpose
  • The implementation accounts for edge cases, avoids subtle logical errors, and handles somewhat rare failure states (e.g. offline mode for mobile, 3rd party being down, etc)

Testing

  • Verify there are automated tests added that meaningfully cover critical behavior and failure cases
  • Code coverage does not go down as result of this change
  • Test suite passes

Security & Data Privacy

  • Verify there is no chance we would accidentally log PII to application logs
  • Verify this addition does not materially affect our security attack surface, and if so it has undergone security review
  • All inputs are sanitized
  • New dependencies are well maintained, have significant justification for being added to the project, and are documented in the Curious open source credit page

Logging/Monitoring

  • Logging is implemented for this change such that you could troubleshoot this feature in production
  • The change/feature is able to be monitored in production

Performance

  • This change does not introduce n+1 queries or other performance issues within our expected scale (e.g. missing indexes on frequently queried columns, frequently updating tables that are accessed often)

Readability

  • All commented out code is removed
  • Debugging code including extraneous log lines are removed
  • Code is easy to understand through naming and structure; comments explain intent or non‑obvious decisions

Change Safety

  • Backend changes are backwards compatible with old clients, or it is well known they are not and a deployment/rollout plan is in place. This include backend changes being compatible with old mobile app versions, as well as applet versioning within Curious.
  • Destructive database migrations are rolled out in stages. For example, renaming a column means adding a new column and migrating the existing data to that columns in one deployment. Then monitoring to ensure that field isn’t used, and finally removing that old column in a separate deployment.

@aws-amplify-us-east-1

Copy link
Copy Markdown

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-2254.d2ccder08v9rmu.amplifyapp.com

@sricharan-varanasi sricharan-varanasi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Approved, Chris

@divbzero

Copy link
Copy Markdown
Contributor Author

Thank you @sricharan-varanasi!

@divbzero
divbzero merged commit e9cd321 into develop Aug 20, 2026
6 checks passed
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.

2 participants