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

Skip to content

removed payment link from navbar#407

Merged
ajinkyaa merged 2 commits intodevelopfrom
remove-payment-link-navbar
May 24, 2022
Merged

removed payment link from navbar#407
ajinkyaa merged 2 commits intodevelopfrom
remove-payment-link-navbar

Conversation

@ajinkyaa
Copy link
Contributor

@ajinkyaa ajinkyaa commented May 24, 2022

Screenshot 2022-05-24 at 5 56 52 PM

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

@ajinkyaa ajinkyaa requested a review from JijoBose May 24, 2022 12:49
@github-actions
Copy link

github-actions bot commented May 24, 2022

Current Code Coverage Percent of this PR:

83.59 %

Files having coverage below 100%

Impacted Files Coverage
/app/controllers/subscriptions_controller.rb 66.67 %
/app/controllers/invoices_controller.rb 57.14 %
/app/models/invoice.rb 96.3 %
/app/models/client.rb 92.68 %
/app/policies/timezone_policy.rb 66.67 %
/app/policies/subscriptions_policy.rb 66.67 %
/app/policies/client_policy.rb 95.45 %
/app/policies/invoice_policy.rb 94.12 %
/app/controllers/invoices/payments_controller.rb 61.11 %
/app/services/report/filters.rb 87.88 %
/app/services/invoice_payment/checkout.rb 43.48 %
/app/services/invoice_payment/pdf_generation.rb 66.67 %
/app/controllers/internal_api/v1/timezones_controller.rb 37.5 %
/app/controllers/internal_api/v1/clients_controller.rb 94.44 %
/app/controllers/internal_api/v1/invoices_controller.rb 94.87 %
/lib/benchmarking/benchmarker.rb 0.0 %
/lib/countries_info.rb 0.0 %

@ajinkyaa ajinkyaa requested a review from apoorv1316 May 24, 2022 15:06
@ajinkyaa ajinkyaa merged commit 96aec1c into develop May 24, 2022
@ajinkyaa ajinkyaa deleted the remove-payment-link-navbar branch May 24, 2022 15:07
vipulnsward pushed a commit that referenced this pull request Feb 15, 2026
removed payment link from navbar (#407)
77231e9
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@apoorv1316 apoorv1316 apoorv1316 approved these changes

@JijoBose JijoBose Awaiting requested review from JijoBose

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

@ajinkyaa @apoorv1316