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

FMWK-466 Add Insert Only Method #158

Merged
merged 3 commits into from
Jul 4, 2024
Merged

Conversation

tim-aero
Copy link
Contributor

If a policy specified either by defaultPolicy or using the policy hierarchy has a RecordExistsAction of CREATE_ONLY, the save method was overwriting this with REPLACE. Changed this to only replace the policy with REPLACE if the passed RecordExistsAction is UPDATE.

If a policy specified either by defaultPolicy or using the policy
hierarchy has a RecordExistsAction of CREATE_ONLY, the save method was
overwriting this with REPLACE. Changed this to only replace the policy
with REPLACE if the passed RecordExistsAction is UPDATE.
@tim-aero tim-aero requested a review from roimenashe May 20, 2024 02:38
@roimenashe
Copy link
Contributor

@tim-aero sent an email with a new design proposal - please review and let's decide together how we should proceed.

@roimenashe roimenashe changed the title Fixed issue with CREATE_ONLY RecordExistsAction FMWK-466 Issue with CREATE_ONLY RecordExistsAction Jun 18, 2024
@roimenashe roimenashe added the bug Something isn't working label Jun 18, 2024
… records exists action

2. Refactor and cleanup (use generics instead of Objects)
3. Add insert() method for both reactive and none-reactive flows and add integration tests
@roimenashe roimenashe changed the title FMWK-466 Issue with CREATE_ONLY RecordExistsAction FMWK-466 Add Insert Only Method Jun 19, 2024
@roimenashe roimenashe requested a review from reugn June 19, 2024 13:58
@tim-aero tim-aero merged commit 0da08a4 into main Jul 4, 2024
6 checks passed
@tim-aero tim-aero deleted the issue-with-insert-only-policy branch July 4, 2024 03:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants