Skip to content

Fix a bunch of issues I introduced with the transition to contexts #11140

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.

Sign up for GitHub

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

Jump to bottom
Merged
nbudin merged 1 commit into from
Dec 12, 2025
Merged

Fix a bunch of issues I introduced with the transition to contexts #11140

nbudin merged 1 commit into from
Dec 12, 2025

Conversation

@nbudin
Copy link
Contributor

@nbudin nbudin commented Dec 12, 2025 •
edited
Loading

This fixes the following bugs:

  • Page crashes when trying to display the sign up form (because reCAPTCHA site key can't be found)
  • 404 when trying to upload files (because rails_direct_uploads_url is undefined)

@nbudin nbudin added bug patch Bumps the patch version number on release labels Dec 12, 2025
@nbudin nbudin merged commit 40501f3 into main Dec 12, 2025
11 checks passed
@nbudin nbudin deleted the fix-recaptcha-error branch December 12, 2025 17:51
@github-actions
Copy link
Contributor

github-actions bot commented Dec 12, 2025

Code Coverage Report: Only Changed Files listed

Package Coverage
Overall Coverage 🟢 50.56%

Minimum allowed coverage is 0%, this run produced 50.56%

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

bug patch Bumps the patch version number on release

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@nbudin