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

Skip to content

invoice edit send functionality#379

Merged
apoorv1316 merged 2 commits intodevelopfrom
invoice-page-send
May 16, 2022
Merged

invoice edit send functionality#379
apoorv1316 merged 2 commits intodevelopfrom
invoice-page-send

Conversation

@ajinkyaa
Copy link
Contributor

@ajinkyaa ajinkyaa commented May 13, 2022

Send Invoice after edit functionality.

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 review from JijoBose and apoorv1316 May 13, 2022 13:00
@apoorv1316
Copy link
Collaborator

apoorv1316 commented May 13, 2022

@ajinkyaa Please upload a loom for this

@github-actions
Copy link

github-actions bot commented May 13, 2022

Current Code Coverage Percent of this PR:

93.5 %

Files having coverage below 100%

Impacted Files Coverage
/app/controllers/payment_settings_controller.rb 66.67 %
/app/controllers/subscriptions_controller.rb 66.67 %
/app/controllers/invoices_controller.rb 63.64 %
/app/models/client.rb 92.68 %
/app/models/invoice.rb 96.3 %
/app/policies/subscriptions_policy.rb 66.67 %
/app/policies/timezone_policy.rb 66.67 %
/app/policies/payment_settings_policy.rb 71.43 %
/app/controllers/invoices/payments_controller.rb 61.11 %
/app/services/invoice_payment/pdf_generation.rb 65.38 %
/app/services/invoice_payment/checkout.rb 43.48 %
/app/controllers/internal_api/v1/payment_settings_controller.rb 45.45 %
/app/controllers/internal_api/v1/timezones_controller.rb 37.5 %
/lib/countries_info.rb 0.0 %

@apoorv1316 apoorv1316 merged commit 63ea7b0 into develop May 16, 2022
@apoorv1316 apoorv1316 deleted the invoice-page-send branch May 16, 2022 11:45
vipulnsward pushed a commit that referenced this pull request Feb 15, 2026
invoice edit send functionality (#379)
20e8079 
* invoice edit send functionality

* react router code added
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@JijoBose JijoBose JijoBose approved these changes

@apoorv1316 apoorv1316 apoorv1316 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.

3 participants

@ajinkyaa @apoorv1316 @JijoBose