Remove remaining Flipcause integration code#871
Open
JoseRodriguez26 wants to merge 1 commit into
Open
Conversation
naomiquinones
requested changes
Jan 13, 2026
Collaborator
naomiquinones
left a comment
naomiquinones
left a comment
There was a problem hiding this comment.
Thanks for working on this. I've noted a couple of small items to change.
| >. | ||
| <a href="{{ url_for('render_donate_page') }}">Donate</a> | ||
| </li> | ||
| <li> |
Collaborator
There was a problem hiding this comment.
We should keep this opening <li> as it has a matching closing tag below.
| <li> | ||
| All credit card donations are processed by Flipcause, our donation | ||
| platform, and we never have access to your credit card information. | ||
| “All credit card donations are processed by a third-party payment processor.” |
Collaborator
There was a problem hiding this comment.
Do we need this line to be in quotes? If not, they could be removed.
Collaborator
|
Hi @JoseRodriguez26 can you kindly address your requests for changes? |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Removes the remaining Flipcause integration code and related assets.
Changes
• Removed Flipcause integration block from templates/base.html
• Deleted unused Flipcause JavaScript (static/js/base.js)
• Deleted Flipcause SCSS partial (static/sass/_flipcause.scss) and removed its import from static/sass/style.scss
• Removed remaining Flipcause references from templates (faqs.html, privacy.html)
Verification
• rg search for Flipcause, flipcause, and fc-* across templates, static/sass, and static/js returns no results.
Implementation Notes / Investigation Summary
When I started working on this issue, my goal was to confirm whether any Flipcause-related code was still actively used in the application or if it could be safely removed.
I began by searching the codebase for Flipcause identifiers (Flipcause, flipcause, fc-, open_window, close_window) across templates, JavaScript, and SCSS files. This investigation showed that the remaining Flipcause HTML elements, JavaScript logic, and SCSS styles were tightly coupled to the old Flipcause integration block that had already been partially removed in earlier work.
Next, I verified whether any donation flows or user-facing functionality still depended on Flipcause. All donation links now route users to the internal donate page or external processors, and no templates referenced the Flipcause widget or modal behavior anymore. This confirmed that the remaining Flipcause code was orphaned.
Based on that, I:
• Removed the remaining Flipcause integration block from base.html
• Deleted the unused Flipcause JavaScript file
• Deleted the Flipcause SCSS partial and removed its import from style.scss
• Cleaned up any remaining references in templates
Finally, I re-ran searches across the repository to confirm that no Flipcause-related identifiers remained and verified that the site continues to function as expected without the legacy code.
This change fully removes the deprecated Flipcause integration and simplifies the codebase by eliminating unused assets and styles.
⸻
Add this one line at the very bottom if it applies:
Closes #865