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

Implemented CRUD_EVENT_NO_CHANGED_FIELDS_SKIP #249

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Implemented CRUD_EVENT_NO_CHANGED_FIELDS_SKIP #249

wants to merge 2 commits into from

Commits on Mar 14, 2023

  1. Implemented CRUD_EVENT_NO_CHANGED_FIELDS_SKIP 

    I experienced issues with replicas when pre_save is used. To avoid this issues, I'd like to silence pre_saves.

I saw we have the flag CRUD_EVENT_NO_CHANGED_FIELDS_SKIP already in documentation but not implemented it. Please let me know if this fix implements CRUD_EVENT_NO_CHANGED_FIELDS_SKIP as it was intended.

Thanks
    @felixcheruiyot
    felixcheruiyot authored Mar 14, 2023
    Configuration menu
    Copy the full SHA
    5b06e48 View commit details
    Browse the repository at this point in the history

  2. Merge pull request #1 from felixcheruiyot/felixcheruiyot-patch-skip-n… 

    …ochange

Implemented CRUD_EVENT_NO_CHANGED_FIELDS_SKIP
    @felixcheruiyot
    felixcheruiyot authored Mar 14, 2023
    Configuration menu
    Copy the full SHA
    9a37b37 View commit details
    Browse the repository at this point in the history