Skip to content
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

Remove partner FAQs - Part 1 #4912

Merged
merged 10 commits into from
Jan 16, 2025
Merged

Conversation

jp524
Copy link
Contributor

@jp524 jp524 commented Jan 4, 2025

Relates to #4546.

Description

Partner FAQs aren't used anymore. The help page for partners shows the bank's name and email instead. (The issue mentions the bank's primary contact name and bank's phone, but I wasn't able to find those.)

The Question model was also modified to remove any reference to banks or partners. Because this change requires removing a column the migration needs to be to done in multiple steps based on the Strong Migrations docs.

The PR with the migration to remove the columns is #4913. This PR should be deployed prior to #4913 being deployed according to Strong Migrations docs.

Type of change

  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Added request tests for /partners/help. Ensured that other tests related to bank FAQ still pass.

Screenshots

Partner Help

Partner Help

Admin All FAQs

Admin All FAQs

Admin New FAQ

Admin New FAQ

@jp524 jp524 changed the title Remove partner FAQs Remove partner FAQs - Part 1 Jan 4, 2025
@cielf
Copy link
Collaborator

cielf commented Jan 6, 2025

@jp54

Hrm. I tried adding a question/answer pair, and it did not appear when I signed in as a bank.

However I'm not sure this is working properly to start with (it's a pretty rarely used feature).

Do you want to take a swing at fixing it?

(The removal from the partner area looks good, FWIW)

@jp524
Copy link
Contributor Author

jp524 commented Jan 6, 2025

@cielf I had a look at the missing Q&A when signed in as a bank user. It looks like only the first page (first 5 Q&A) get loaded so the new Q&A don't appear until a search is made. I'll work on a fix.

@jp524
Copy link
Contributor Author

jp524 commented Jan 7, 2025

@cielf The bank FAQ page shows all the Q&A now.
Screenshot 2025-01-06 at 17 44 39

@cielf cielf requested a review from dorner January 8, 2025 01:42
@cielf
Copy link
Collaborator

cielf commented Jan 8, 2025

LGTM from a functional pov. @dorner -- what say you on the tech side?

Copy link
Collaborator

@dorner dorner left a comment

Choose a reason for hiding this comment

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

Tiny request - otherwise looks good!

spec/requests/partners/helps_requests_spec.rb Outdated Show resolved Hide resolved
@jp524 jp524 requested a review from dorner January 10, 2025 23:06
Copy link
Collaborator

@dorner dorner left a comment

Choose a reason for hiding this comment

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

Looks good - thanks!

@dorner dorner merged commit c00f0f8 into rubyforgood:main Jan 16, 2025
11 checks passed
@jp524 jp524 deleted the 4546-remove-partner-faq branch January 16, 2025 00:33
coalest added a commit to coalest/human-essentials that referenced this pull request Jan 17, 2025
dorner pushed a commit that referenced this pull request Jan 17, 2025
Copy link
Contributor

@jp524: Your PR Remove partner FAQs - Part 1 is part of today's Human Essentials production release: 2025.01.19.
Thank you very much for your contribution!

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