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.

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

[ADD] New module edi_sale_forward_request #63

Open
wants to merge 5 commits into
base: wip_13.0
Choose a base branch
from
Open

[ADD] New module edi_sale_forward_request #63

wants to merge 5 commits into from

Conversation

RHODOK
Copy link
Contributor

@RHODOK RHODOK commented Apr 14, 2020

No description provided.

@RHODOK RHODOK changed the base branch from master to wip_13.0 April 16, 2020 02:13
Joe and others added 5 commits April 16, 2020 03:13
This test executes a document with a patch that causes failure in
order to test the rollback functionality.
Clearing the cache and fields that need to be recalculated after a
failure rollback will prevent further failures from recalculations
or accessing cached objects
    Recent changes to fix issues with comparators has introduced a
    new edge case. Two identical values may be evaluated as different
    when part of them (while being the same value) needs to be
    constructed as another new record. Example: partner title record
    has not yet been constructed and the same partner is attempted to
    be created twice, this results in the partner being identified as
    different causing an attempt at creating the parnter twice, the
    unique constraint for a partner then causes the creation of a
    partner to be invalidated.

Signed-off-by: Robert Smith <[email protected]>
When a partner is created and no title of the type already exists
a models.NewId is used, this causes the same person ingested twice
to be incorrectly treated as separate entities.

Signed-off-by: Robert Smith <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants