anitab-org / stem-diverse-tv

STEM Diverse TV is a project which gather and provide inspiring, motivating, informative, educational and supportive videos about diversity in STEM. This is a backend which provide content for multiple frontend applications build with Applicaster Zapp platform.
https://stem-diverse-tv.herokuapp.com/
GNU General Public License v3.0
13 stars 49 forks source link

Update author controller to return object on post #261

Open FarzamMohammadi opened 2 years ago

FarzamMohammadi commented 2 years ago

Description

Author POST API, which originally returned a "Author added successfully created" message has been updated to return the created author object instead.

Fixes # [123]

Type of Change:

Code/Quality Assurance Only

How Has This Been Tested?

Describe the tests you ran to verify your changes. Provide instructions or GIFs so we can reproduce. List any relevant details for your test.

Checklist:

Delete irrelevant options.

Code/Quality Assurance Only

welcome[bot] commented 2 years ago

Hello there!πŸ‘‹ Welcome to the project!πŸ’–

Thank you and congratsπŸŽ‰ for opening your first pull request.✨ AnitaB.org is an inclusive community, committed to creating a safe and positive environment.🌸Please adhere to our Code of Conduct and Contributing Guidelines.πŸ™Œ.We will get back to you as soon as we can.πŸ˜„

Feel free to join us on AnitaB.org Open Source Zulip Community.πŸ’– We have different streams for each active repository for discussions.✨ Hope you have a great time there!πŸ˜„

FarzamMohammadi commented 1 year ago

Anyone wants to merge this? 😒

NenadPantelic commented 1 year ago

@FarzamMohammadi can you please check these two comments I have left recently? Also, check how the frontend processes this response (I haven't opened it in a while, so I'm not sure whether we have this endpoint covered there :sweat_smile: ).

FarzamMohammadi commented 1 year ago

@FarzamMohammadi can you please check these two comments I have left recently? Also, check how the frontend processes this response (I haven't opened it in a while, so I'm not sure whether we have this endpoint covered there πŸ˜… ).

What comments?