Skip to content
This repository was archived by the owner on Dec 7, 2023. It is now read-only.

Conversation

@Nydauron
Copy link
Contributor

@Nydauron Nydauron commented Jun 23, 2022

Closes #8. Ports the current database helper functions to use mongo/mongo-go-driver

Still need to:

  • Add timeouts to context (add ctx property to Database struct)
  • Fix multi-document CRUDs from data races (implement ACID transactions)
  • Port e2e tests over to use new driver
  • Remove all references to mgo

@Nydauron Nydauron marked this pull request as ready for review August 12, 2022 23:43
@Nydauron
Copy link
Contributor Author

This PR changes nothing to any of the endpoints. All the endpoints should behave identically prior to the driver upgrade.

Throughout implementing dtm into the API, endpoints will be changed to use transactions if need be.

Copy link

@ashayp22 ashayp22 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks great

@Nydauron Nydauron merged commit 0e76be3 into HackIllinois:staging Oct 7, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

gopkg.in/mgo.v2 is now unmaintained

2 participants