acmucsd / membership-portal

REST API for the UC San Diego ACM chapter's membership portal.
https://members.acmucsd.com
Mozilla Public License 2.0
17 stars 5 forks source link

Fix transaction bug for batch social media writes #384

Closed nik-dange closed 6 months ago

nik-dange commented 8 months ago

Description

transaction error when adding multiple social media URLs at the same time

Changes

Type of Change

If you've selected Patch, Minor, or Major as your change type, make sure to bump the version before merging in package.json!

Testing

I have tested that my changes fully resolve the linked issue ...

Checklist

Screenshots

Please include a screenshot of your Postman testing passing successfully.

github-actions[bot] commented 8 months ago

Thanks for contributing! If you've made changes to the API's functionality, please make sure to bump the package version—see this guide to semantic versioning for details—and document those changes as appropriate.

nik-dange commented 8 months ago

TODO: similar fix for edit/delete depending on what the user functionality is looking like

nik-dange commented 7 months ago

Marking as something to revisit in the future (in case this code is useful), when we have a discussion about standardizing API request formats