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

Include mandatory fields into sync #9

Closed
koltyakov opened this issue Feb 24, 2023 · 1 comment
Closed

Include mandatory fields into sync #9

koltyakov opened this issue Feb 24, 2023 · 1 comment
Assignees
Labels
enhancement New feature or request

Comments

@koltyakov
Copy link
Owner

koltyakov commented Feb 24, 2023

Some fields should always be included no matter were they requested explicitly or not.

Such fields are:

  • ID
  • Created
  • Modified
  • Author
  • Editor
@koltyakov koltyakov self-assigned this Feb 24, 2023
@koltyakov koltyakov added the enhancement New feature or request label Feb 25, 2023
@koltyakov
Copy link
Owner Author

The mandatory fields are added. They automatically injected into the request.

Also, the code is refactored a bit to rely in explicit props instead of making a try using all and ignoring service fields.

This design decision is driven by the aspect, that it's not always only List fields can be synced, but also a nested object model props which do not exist in List/Fields metadata. E.g. file props or a lookup prop.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant