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

PaymentController: added few changes to help with the redirections #97

Merged
merged 1 commit into from
Jul 26, 2024

Conversation

Jovz19200
Copy link
Collaborator

@Jovz19200 Jovz19200 commented Jul 25, 2024

What does PR Do?

  • help us with the redirection after payment

Copy link

railway-app bot commented Jul 25, 2024

This PR was not deployed automatically as @Jovz19200 does not have access to the Railway project.

In order to get automatic PR deploys, please add @Jovz19200 to the project inside the project settings page.

@Jovz19200 Jovz19200 force-pushed the fix-Payment-Redirect branch from 3933f1c to 4324851 Compare July 26, 2024 09:03
@Jovz19200 Jovz19200 force-pushed the fix-Payment-Redirect branch from 4324851 to c2f5f45 Compare July 26, 2024 10:46
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 72.61%. Comparing base (2e4abc4) to head (c2f5f45).

Additional details and impacted files
@@            Coverage Diff             @@
##              dev      #97      +/-   ##
==========================================
- Coverage   73.08%   72.61%   -0.47%     
==========================================
  Files         130      130              
  Lines        2370     2370              
  Branches      302      301       -1     
==========================================
- Hits         1732     1721      -11     
- Misses        638      649      +11     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@teerenzo teerenzo merged commit 13f184d into dev Jul 26, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants