Skip to content
This repository has been archived by the owner on Sep 3, 2024. It is now read-only.

Hotfix/mailing list #22

Merged
merged 5 commits into from
May 16, 2023
Merged

Hotfix/mailing list #22

merged 5 commits into from
May 16, 2023

Conversation

VirginiaDooley
Copy link
Contributor

@VirginiaDooley VirginiaDooley commented May 1, 2023

Users receive a 500 error when trying to sign up for the mailing list.
Closes https://democracy-club-gp.sentry.io/share/issue/f2465a9aa44148678fbedd491696006d/
Ref https://trello.com/c/umVpqIL6/3345-fix-mailing-list-signup

This work refactors definition of the source_url ensuring it's not blank. This has been installed locally in Website with success.

Screenshot 2023-05-01 at 5 12 37 PM

Tasks

Preview Give feedback

@VirginiaDooley VirginiaDooley marked this pull request as ready for review May 1, 2023 16:15
@VirginiaDooley
Copy link
Contributor Author

Fix mailing list signup

@VirginiaDooley VirginiaDooley force-pushed the hotfix/mailing-list branch from aa2c752 to ca271d9 Compare May 2, 2023 12:34
@VirginiaDooley VirginiaDooley force-pushed the hotfix/mailing-list branch from d78d534 to a38d9b0 Compare May 2, 2023 12:50
@VirginiaDooley VirginiaDooley requested a review from symroe May 2, 2023 12:52
@VirginiaDooley VirginiaDooley requested a review from Bekabyx May 10, 2023 14:27
Copy link

@Bekabyx Bekabyx left a comment

Choose a reason for hiding this comment

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

No longer getting an error on this locally, looks like it's sorted :)

@VirginiaDooley VirginiaDooley merged commit ec137bf into master May 16, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants