Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[SECURITY]Remove stripe secret key#365

Merged
apoorv-mishra merged 1 commit intodevelopfrom
remove-secret-key
May 9, 2022
Merged

[SECURITY]Remove stripe secret key#365
apoorv-mishra merged 1 commit intodevelopfrom
remove-secret-key

Conversation

@apoorv-mishra
Copy link
Contributor

@apoorv-mishra apoorv-mishra commented May 9, 2022

Notion card

NA

Summary

Remove test stripe secret key from .env.example

Preview

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to
    not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

Checklist:

  • I have manually tested all workflows
  • I have performed a self-review of my own code
  • I have added automated tests for my code

@github-actions
Copy link

github-actions bot commented May 9, 2022

Current Code Coverage Percent of this PR:

95.11 %

Files having coverage below 100%

Impacted Files Coverage
/app/controllers/subscriptions_controller.rb 66.67 %
/app/controllers/invoices_controller.rb 63.64 %
/app/models/client.rb 92.5 %
/app/models/invoice.rb 96.15 %
/app/policies/timezone_policy.rb 66.67 %
/app/policies/subscriptions_policy.rb 66.67 %
/app/services/application_service.rb 66.67 %
/app/controllers/invoices/payments_controller.rb 61.11 %
/app/services/invoice_payment/checkout.rb 43.48 %
/app/controllers/internal_api/v1/timezones_controller.rb 37.5 %
/lib/countries_info.rb 0.0 %

@apoorv-mishra apoorv-mishra merged commit a093c8d into develop May 9, 2022
@apoorv-mishra apoorv-mishra deleted the remove-secret-key branch May 9, 2022 05:55
vipulnsward pushed a commit that referenced this pull request Feb 15, 2026
Remove stripe secret key (#365)
79d4d3e
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@akhilgkrishnan akhilgkrishnan akhilgkrishnan approved these changes

Assignees

No one assigned

Labels

PR: merged

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@apoorv-mishra @akhilgkrishnan