Skip to content

Conversation

@powersurge360
Copy link
Contributor

Link to pivotal/JIRA issue

https://codeforamerica.atlassian.net/browse/FYST-2299

Is PM acceptance required? (delete one)

  • Yes - don't merge until JIRA issue is accepted!

Reminder: merge main into this branch and get green tests before merging to main

What was done?

  • Removed date and state specific logic for when FAQ items are visible.

How to test?

  • Visit fyst faq page
  • See that all items are visible, despite the end date being set to the past.

Screenshots (for visual changes)

  • Before
  • After

@github-actions
Copy link

Heroku app: https://gyr-review-app-6108-aa36e3c22795.herokuapp.com/
View logs: heroku logs --app gyr-review-app-6108 (optionally add --tail)

Copy link
Contributor

@DrewProebstel DrewProebstel left a comment

Choose a reason for hiding this comment

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

Looks great thanks for doing this!

@powersurge360 powersurge360 merged commit 3ebc688 into main Nov 19, 2025
8 checks passed
@powersurge360 powersurge360 deleted the FYST-2299/update-faqs-for-pya branch November 19, 2025 17:28
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