forked from ontoportal/goo
-
Notifications
You must be signed in to change notification settings - Fork 4
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
Feature: Add models on update callbacks #31
Merged
syphax-bouazzouni
merged 5 commits into
development
from
feature/add-models-on-update-callbacks
Mar 16, 2023
Merged
Feature: Add models on update callbacks #31
syphax-bouazzouni
merged 5 commits into
development
from
feature/add-models-on-update-callbacks
Mar 16, 2023
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
34 tasks
syphax-bouazzouni
added a commit
that referenced
this pull request
Jul 8, 2023
…date-callbacks Feature: Add models on update callbacks
syphax-bouazzouni
added a commit
that referenced
this pull request
Sep 5, 2023
…date-callbacks Feature: Add models on update callbacks
syphax-bouazzouni
added a commit
that referenced
this pull request
Sep 5, 2023
…date-callbacks Feature: Add models on update callbacks
syphax-bouazzouni
added a commit
that referenced
this pull request
Dec 4, 2023
…scheme file and callbacks) (#45) * Merge pull request #23 from ontoportal-lirmm/feature/add-property-datatype-dsl Feature: add property data type dsl * Merge pull request #26 from ontoportal-lirmm/refactor/resources-validators Refactor: resources validators * Merge pull request #27 from ontoportal-lirmm/feature/implement-new-validators Feature: Implement new validators * Merge pull request #28 from ontoportal-lirmm/feature/implement-new-validators Feature: Implement instance proc validators * Merge pull request #30 from ontoportal-lirmm/feature/add-model-scheme-yml-file Feature/add model scheme yml file * Merge pull request #31 from ontoportal-lirmm/feature/add-models-on-update-callbacks Feature: Add models on update callbacks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Requirement
This PR adds the possibility to add an onUpdate callback to a model attribute
For example
Will ensure that each time we update firt_name or last_name we call the method
update_name
and make that name will always be firt_name + last_nameChanges
add onUpdate callback tests (c83687e)
implement enforce_callback to run an attribute callback (e4979ff)
move the attribute default callback to the save method (c7f7092)
implement onUpdate DSL in the ressource settings (f81b15f)
call to the attributes onUpdate callback in the save method (714f085)