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.

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

Issue with business users #911

Closed
wants to merge 32 commits into from
Closed

Issue with business users #911

wants to merge 32 commits into from

Conversation

distich
Copy link

@distich distich commented Nov 18, 2024 •
edited
Loading

No description provided.

distich added 30 commits November 16, 2024 12:31
@distich
fix: Allow unsafe redirect to external host in notifications controller
8adcee3
@distich
feat: Enable SSL in Puma configuration for local development
b1a0864
@distich
fix: Update comment to clarify SSL configuration in puma.rb
15f359f
@distich
feat: Configure Puma to use SSL with verify_mode set to none
9b855dd
@distich
fix: Update Puma SSL binding to 0.0.0.0 on port 3001
a798982
@distich
feat: Update Procfile.dev to use SSL port 3001 for Rails server
98440e8
@distich
chore: Update Procfile.dev to enable SSL for the Rails server
292d077
@distich
fix: Remove allow_other_host option from redirect_to in notifications…
0ef10d5 
…_controller
@distich
fix: Update default_url_options to use correct host and port for URLs
b1620a3
@distich
fix: Update default_url_options to use request settings for URLs
86e4494
@distich
fix: Correct URL construction in NotificationsController redirect
4839e03
@distich
fix: Correct URL construction in NotificationsController redirect logic
e7a1501
@distich
fix: Use correct base URL for redirect in NotificationsController
a0927d1
@distich
fix: Correct URL construction for redirection in NotificationsController
63c3d9e
@distich
fix: Update URL redirection logic in NotificationsController to use b…
96bfa65 
…ase URL
@distich
fix: Correct URL construction in NotificationsController redirection …
1aa094e 
…logic
@distich
fix: Ensure redirection only occurs for valid notification URLs
a7956b0
@distich
fix: Redirect to specific notification message instead of notificatio…
2b902a0 
…ns index
@distich
fix: Redirect to the correct conversation path in NotificationsContro…
4b91947 
…ller
@distich
fix: Resolve undefined method by accessing conversation ID correctly
383f6b3
@distich
fix: Update Stripe webhook port to match application port 3001
fbde076
@distich
fix: Change application and Stripe webhook ports back to 3000
7745063
@distich
feat: Add middleware to redirect HTTP traffic to HTTPS in development…
892a04c 
… environment
@distich
fix: Replace Rack::SSL with config.force_ssl in development environment
6578f37
@distich
fix: Handle missing conversation method in NotificationsController
5cee038
@distich
fix: Handle missing conversation ID in notifications redirect logic
7cacf40
@distich
chore: Add logging for conversation redirection in NotificationsContr…
c96ef36 
…oller
@distich
fix: Add logging for missing conversation or conversation ID in notif…
6da5e96 
…ications controller
@distich
fix: Ensure valid conversation ID before redirecting in Notifications…
a4a8b49 
…Controller
@distich
fix: Handle missing conversation ID in notifications redirection logic
764f3b9
@distich distich closed this Nov 18, 2024
@distich distich deleted the Issue-with-business-users branch November 18, 2024 15:56
@distich distich restored the Issue-with-business-users branch November 18, 2024 16:15
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
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@distich