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

Describe semantics of etags within a delta payload. #1958

Merged
merged 17 commits into from
Jul 31, 2024
Merged

Describe semantics of etags within a delta payload. #1958

merged 17 commits into from
Jul 31, 2024

Conversation

mikepizzo
Copy link
Contributor

Fixes #358

HeikoTheissen
HeikoTheissen previously requested changes Jun 20, 2024
View reviewed changes
odata-protocol/11.4 Data Modification.md Outdated Show resolved Hide resolved
@mikepizzo
Apply review feedback.
0b227f3 
Include description of etags for replacing versus updating a collection.
ralfhandl
ralfhandl reviewed Jul 26, 2024
View reviewed changes
Copy link
Contributor

@ralfhandl ralfhandl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

Only some word smithing.

odata-protocol/1 Introduction.md Outdated Show resolved Hide resolved
odata-protocol/11.4 Data Modification.md Outdated Show resolved Hide resolved
odata-protocol/11.4 Data Modification.md Outdated Show resolved Hide resolved
odata-protocol/11.4 Data Modification.md Outdated Show resolved Hide resolved
odata-protocol/11.4 Data Modification.md Outdated Show resolved Hide resolved
odata-protocol/11.4 Data Modification.md Outdated Show resolved Hide resolved
mikepizzo and others added 9 commits July 30, 2024 17:23
@mikepizzo @ralfhandl
Update odata-protocol/11 Data Service Requests.md
c866f02 
Co-authored-by: Ralf Handl <[email protected]>
@mikepizzo @ralfhandl
Update odata-protocol/1 Introduction.md
b33cf86 
Co-authored-by: Ralf Handl <[email protected]>
@mikepizzo @ralfhandl
Update odata-protocol/11.4 Data Modification.md
625fbbd 
Co-authored-by: Ralf Handl <[email protected]>
@mikepizzo @ralfhandl
Update odata-protocol/11.4 Data Modification.md
affe371 
Co-authored-by: Ralf Handl <[email protected]>
@mikepizzo @ralfhandl
Update odata-protocol/11.4 Data Modification.md
f16b751 
Co-authored-by: Ralf Handl <[email protected]>
@mikepizzo @ralfhandl
Update odata-protocol/11.4 Data Modification.md
6238f79 
Co-authored-by: Ralf Handl <[email protected]>
@mikepizzo @ralfhandl
Update odata-protocol/11.4 Data Modification.md
06085b2 
Co-authored-by: Ralf Handl <[email protected]>
@mikepizzo
Apply review feedback
2128853
@mikepizzo
Merge branch 'main' into odata358
856ccfb
ralfhandl
ralfhandl previously approved these changes Jul 31, 2024
View reviewed changes
Copy link
Contributor

@ralfhandl ralfhandl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One remark, which can be addressed in this PR or via a new issue.

odata-protocol/11.4 Data Modification.md Show resolved Hide resolved
@mikepizzo mikepizzo merged commit b4c4fcc into main Jul 31, 2024
1 check passed
@mikepizzo mikepizzo deleted the odata358 branch July 31, 2024 15:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ralfhandl ralfhandl ralfhandl approved these changes

@HeikoTheissen HeikoTheissen Awaiting requested review from HeikoTheissen

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Clarify semantics of etag in delta requests/responses
3 participants
@mikepizzo @ralfhandl @HeikoTheissen