Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

IDEMPIERE-5915 Generate Invoices (manual) > RMA => AR Credit Memo created with system date instead of "Date Invoiced" #2097

Merged
merged 2 commits into from Nov 9, 2023
Merged

IDEMPIERE-5915 Generate Invoices (manual) > RMA => AR Credit Memo created with system date instead of "Date Invoiced" #2097

merged 2 commits into from Nov 9, 2023

Conversation

etantg
Copy link
Contributor

@etantg etantg commented Nov 9, 2023

https://idempiere.atlassian.net/browse/IDEMPIERE-5915

Generate Invoices (manual)

  1. Go to the 'Generate Invoices (manual)' window
  2. Select an 'RMA' record
  3. Click on the 'Generate Invoices for RMA' button
  4. In the 'Generate Invoices for RMA' dialog, enter the 'Date Invoiced'
  5. Click on the 'OK' button
  6. Zoom to the created invoice (i.e. AR Credit Memo)
  7. The 'Date Invoiced' and 'Account Date' should be the same 'Date Invoiced' entered in Step 4

Generate Shipments (manual)

  1. Go to the 'Generate Shipments (manual)' window
  2. Select an 'RMA' record
  3. Click on the 'Generate Shipments for Vendor RMA' button
  4. In the 'Generate Shipments for Vendor RMA' dialog, enter the 'Movement Date'
  5. Click on the 'OK' button
  6. Zoom to the created shipment (i.e. MM Vendor Return)
  7. The 'Movement Date' and 'Account Date' should be the same 'Movement Date' entered in Step 4

@etantg
IDEMPIERE-5915 Generate Invoices (manual) > RMA => AR Credit Memo cre…
1d4d767 
…ated with system date instead of "Date Invoiced"
@etantg
IDEMPIERE-5915 Generate Invoices (manual) > RMA => AR Credit Memo cre…
76f4cb1 
…ated with system date instead of "Date Invoiced"
CarlosRuiz-globalqss
CarlosRuiz-globalqss approved these changes Nov 9, 2023
View reviewed changes
Copy link
Collaborator

@CarlosRuiz-globalqss CarlosRuiz-globalqss left a comment

Choose a reason for hiding this comment

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

looks good - new unit tests failed before change and passed successfully after

@CarlosRuiz-globalqss CarlosRuiz-globalqss merged commit d068b19 into idempiere:master Nov 9, 2023
4 checks passed
CarlosRuiz-globalqss pushed a commit that referenced this pull request Nov 10, 2023
@etantg @CarlosRuiz-globalqss
IDEMPIERE-5915 Generate Invoices (manual) > RMA => AR Credit Memo cre…
1434672 
…ated with system date instead of "Date Invoiced" (#2097)

* IDEMPIERE-5915 Generate Invoices (manual) > RMA => AR Credit Memo created with system date instead of "Date Invoiced"

* IDEMPIERE-5915 Generate Invoices (manual) > RMA => AR Credit Memo created with system date instead of "Date Invoiced"
freddyheredia4 pushed a commit to freddyheredia4/idempiere that referenced this pull request Feb 28, 2024
@etantg @freddyheredia4
IDEMPIERE-5915 Generate Invoices (manual) > RMA => AR Credit Memo cre…
5959405 
…ated with system date instead of "Date Invoiced" (idempiere#2097)

* IDEMPIERE-5915 Generate Invoices (manual) > RMA => AR Credit Memo created with system date instead of "Date Invoiced"

* IDEMPIERE-5915 Generate Invoices (manual) > RMA => AR Credit Memo created with system date instead of "Date Invoiced"
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@CarlosRuiz-globalqss CarlosRuiz-globalqss CarlosRuiz-globalqss approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@etantg @CarlosRuiz-globalqss