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

Skip to content

Fixed the issue in closing the send invoice modal#517

Merged
supriya3105 merged 1 commit intodevelopfrom
fix-save-invoice
Jun 24, 2022
Merged

Fixed the issue in closing the send invoice modal#517
supriya3105 merged 1 commit intodevelopfrom
fix-save-invoice

Conversation

@gowsik-ragunath
Copy link
Contributor

@gowsik-ragunath gowsik-ragunath commented Jun 23, 2022

Notion card

https://www.notion.so/saeloun/Send-invoice-is-not-working-from-edit-invoice-page-a35c37f20fb5494f8fef5e9f95d8bd9f

Summary

Fixed the issue in closing the send invoice modal and displaying the toaster message.

Invoice status will remain in Draft status until the invoice mail is sent then it will be updated to Sent status.

Preview

https://www.loom.com/share/c77d12d8412f4850b7ecb21f8b77e826

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

Copy link
Contributor

@sudeeptarlekar sudeeptarlekar 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 e9d4042 into develop Jun 24, 2022
@supriya3105 supriya3105 deleted the fix-save-invoice branch June 24, 2022 05:42
vipulnsward pushed a commit that referenced this pull request Feb 15, 2026
Fixed the issue in closing the send invoice modal and displaying the toaster

Co-authored-by: Gowsik <[email protected]>
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.

4 participants