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

Event nav + ticket request form #103

Merged
merged 8 commits into from
Apr 30, 2024
Merged

Event nav + ticket request form #103

merged 8 commits into from
Apr 30, 2024

Conversation

kigster
Copy link
Collaborator

@kigster kigster commented Apr 25, 2024

This very large PR includes the following changes:

Completed

  • Upgrading gems
  • Fixing buttons and links
  • Switching Event sub-nav to use tabs instead of buttons
  • Adding #after_sign_in_path_for() and #set_redirect_path helpers to ApplicationController
  • This allows using login/register partial within the context of the ticket request form
  • Redirecting from ticket request form to the home if tickets are not on sale yet
  • Fixing add admin/demote admin on events admin edit page
  • Fixing the ticket request form to work and submit.

Temporarily

  • Removed the payments/stripe JS until we redo the integration

Remaining

Wireing up application mailer to the ticket requests.

@kigster kigster requested review from beingmattlevy and a team April 29, 2024 14:58
@kigster kigster changed the title WIP: devise turbo forms around login/logout Event nav + ticket request form Apr 29, 2024
@kigster kigster merged commit f913f35 into main Apr 30, 2024
3 checks passed
@kigster kigster deleted the kig/devise-nav branch April 30, 2024 19:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants