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

feat: add datastore class to standardize things #8

Merged
merged 25 commits into from
May 5, 2021

Conversation

ndvo
Copy link
Contributor

@ndvo ndvo commented May 3, 2021

  • refactor tests to avoid using rewire
  • replace Foxy node API for Foxy SDK
  • create a FoxyWebhook module to concentrate tasks related to processing requests from Foxy
  • create a Datastore class to push to a standardization of the interface of the thirdparty services we are integrating with
  • create basic tests for all functions

@ndvo ndvo requested a review from brettflorio May 3, 2021 23:11
@ndvo
Copy link
Contributor Author

ndvo commented May 3, 2021

There is still much to polish, but this should be a step forward. =)

@brettflorio brettflorio changed the title Polish/datastores feat: add datastore class to standardize things May 5, 2021
@brettflorio brettflorio merged commit 5261a5c into main May 5, 2021
@brettflorio brettflorio deleted the polish/datastores branch May 5, 2021 06:10
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.

2 participants