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

Skip to content

Multiline modal invoice#356

Merged
supriya3105 merged 12 commits intodevelopfrom
multiline-modal-invoice
May 5, 2022
Merged

Multiline modal invoice#356
supriya3105 merged 12 commits intodevelopfrom
multiline-modal-invoice

Conversation

@ajinkyaa
Copy link
Contributor

@ajinkyaa ajinkyaa commented May 4, 2022

Multiline Item modal.
Screenshot 2022-05-04 at 11 20 23 AM

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 4, 2022

Current Code Coverage Percent of this PR:

96.55 %

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 95.83 %
/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 %

@supriya3105
Copy link
Contributor

@ajinkyaa video and changes made to this PR are not mentioned on this PR. @apoorv1316 please test this as per our yesterday's observations .

@supriya3105 supriya3105 requested review from JijoBose and apoorv1316 May 5, 2022 11:33
Copy link
Contributor

@JijoBose JijoBose left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@supriya3105 supriya3105 merged commit 28d37ea into develop May 5, 2022
@supriya3105 supriya3105 deleted the multiline-modal-invoice branch May 5, 2022 14:47
vipulnsward pushed a commit that referenced this pull request Feb 15, 2026
* invoice page multiline

* multiline modal generate invoice

* code reverted

* search button aligned

* comments resolved

* lint fixes

* lint fix

* issues resolved

* removed unwanted files
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants