-
Notifications
You must be signed in to change notification settings - Fork 365
Create Collection Endpoint (User Option + Auto-Creation) #1209
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
Conversation
|
@Vriti62 is attempting to deploy a commit to the avdheshvarshney's projects Team on Vercel. A member of the Team first needs to authorize it. |
✅ Deploy Preview for code-a2z canceled.
|
|
Thank you for submitting your pull request! We'll review it as soon as possible. For further communication, join our discord server https://discord.gg/tSqtvHUJzE. |
Avdhesh-Varshney
left a comment
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.
I think both of the PRs having same code. OK, perfect to see the working /create-collection api 👍
In next PRs, are you targetting for get request of all projects & collections?
Actually both these PRs are raised around the same time, with similar code cause they are linked. This PR has some fixes for the concerned endpoint, createNewCollection My next PR is including: I'm hoping this PR is up to the mark and no changes are required!! |
|
hey! |
Do all unresolved changes of #1207 PR here!
|
Avdhesh-Varshney
left a comment
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.
All good. Approved 🎉
Closes: #1197
Linked to parent issue : #1180
Project Published URL 🔗
Backend-only change. Tested locally using Postman. Example API:
Describe the add-ons or changes you've made 📃
Screenshots 📷
Type of change ☑️
What sort of change have you made:
Note to reviewers 📄
Open for any feedback or changes always!!