-
Notifications
You must be signed in to change notification settings - Fork 8
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
OFS-285: changes regarding bulk action for FS entities #57
Conversation
…e_py into feature/OFS-285
.objects \ | ||
.filter( | ||
Q(date_valid_from=None) | Q(date_valid_from__lte=now), | ||
Q(date_valid_to=None) | Q(date_valid_to__gte=now) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Q(date_valid_from=None) | Q(date_valid_from__lte=now),
change it to
Q(date_valid_from__lte=now),
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok
return_objects: bool = False): | ||
""" | ||
dedicated extended mutaion from flter decorator dedicated for Formal Sector entities. See doc string for | ||
mutation_on_uuids_from_filter to read more how this works. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please mention GistoryModel BusinessHistoryModel instead for FS (I hope that those model will be used in the future by other)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok, I can rename also this decorator to not use "fs". I will rename to "history_model"
core/gql/gql_mutations/__init__.py
Outdated
@@ -1,3 +1,3 @@ | |||
from .input_types import * | |||
from .exceptions import ObjectNotExistException, MutationValidationException | |||
from .mutation_by_filter import mutation_on_uuids_from_filter | |||
from .mutation_by_filter import mutation_on_uuids_from_filter, mutation_on_uuids_from_filter_fs |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remane it _history iso _fs
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok
TICKET: https://openimis.atlassian.net/browse/OFS-285