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

Tutorials for making crud operations using Postman #8

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

priyanshunayan
Copy link
Member

@priyanshunayan priyanshunayan commented Jan 25, 2021

@priyanshunayan
Copy link
Member Author

priyanshunayan commented Jan 25, 2021

I was not able to make a POST request to the collection endpoint. Can someone provide me some hints on how to do that?
I am getting the following response:

{
    "message": "The method is not allowed for the requested URL."
}

@Mec-iS
Copy link
Collaborator

Mec-iS commented Jan 25, 2021

@priyanshunayan is updating a collection allowed in Hydra?

@priyanshunayan
Copy link
Member Author

I see that hydra:Collection is subclass of hydra:Resource. We can update hydra:Resource, so I guess, we should also be able to update the collection as well. How do you suggest changing the members (like removing or adding members) of the particular collection if an update is not allowed?

@Mec-iS
Copy link
Collaborator

Mec-iS commented Jan 25, 2021

@chrizandr with other contrivutirs was working on this at some point. please search among issues and PRs

@chrizandr
Copy link
Member

chrizandr commented Jan 29, 2021

It is allowed as long as the entire collection is sent in the request. Should be working, @sameshl has added this.
Can you check if POST operation is allowed in the API doc you're using?
We still need to see how we can update only some items in the collection without the need of sending the entire collection in the request body.

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

Successfully merging this pull request may close these issues.

3 participants