-
Notifications
You must be signed in to change notification settings - Fork 0
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
Feat implement WorkItemAttachment and add relations to WorkItem #13
Conversation
…em attachments on creation anymore
…efore checksum calculation for more reliability
…compare workitems, so that we use the checksum if it is present and calculate it, if it is not present
@ewuerger TBD: Do we want to include the ID in the checksum calculation? There is one big downside of this: When we initially post a WorkItem, we don't have the ID, so we would have to patch a WorkItem always at least once after creation. |
I think it makes only sense to include data which can change in the checksum. IDs are stable, on the other side the |
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.
One tiny thing. Then: LGTM.
This PR introduces the following features:
attachments
orlinkedWorkItems
to the fields parameter to get them