Skip to content

Append patch new document#1745

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
Merged
bourgeoa merged 29 commits into from
Dec 17, 2023
Merged

Append patch new document #1745

bourgeoa merged 29 commits into from
Dec 17, 2023

Conversation

@bourgeoa
Copy link
Member

@bourgeoabourgeoa commented Nov 30, 2023

This allow to PATCH a new document with Append only.

@bourgeoabourgeoa linked an issue Nov 30, 2023 that may be closed by this pull request
PATCH do not allow to create new file with APPEND #1731
Closed
@csarven
Copy link
Member

csarven commented Nov 30, 2023

Not sure I understand. To create a new resource (C/R) with PATCH (as with PUT) requires Append (or Write) on C/ and Write on C/R.

This was referenced Nov 30, 2023
Open
Open
TallTed
TallTed reviewed Nov 30, 2023
View reviewed changes
TallTed
TallTed reviewed Nov 30, 2023
View reviewed changes
TallTed
TallTed reviewed Nov 30, 2023
View reviewed changes
@bourgeoa
Copy link
MemberAuthor

bourgeoa commented Nov 30, 2023

Not sure I understand. To create a new resource (C/R) with PATCH (as with PUT) requires Append (or Write) on C/ and Write on C/R.

A very fat finger .....

1 similar comment
@bourgeoa
Copy link
MemberAuthor

bourgeoa commented Nov 30, 2023

Not sure I understand. To create a new resource (C/R) with PATCH (as with PUT) requires Append (or Write) on C/ and Write on C/R.

A very fat finger .....

bourgeoaand others added 17 commits November 30, 2023 20:41
@bourgeoabourgeoa merged commit acf53e5 into mainDec 17, 2023
@bourgeoabourgeoa deleted the appendPatchNewDocument branch February 11, 2024 16:38
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@TallTedTallTedTallTed left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

PATCH do not allow to create new file with APPEND

4 participants

@bourgeoa@csarven@TallTed