Skip to content

feat(spec): Add API implementation#32

Merged
josecolella merged 9 commits intomainfrom
jc-add-api-object
Nov 14, 2022
Merged

feat(spec): Add API implementation#32
josecolella merged 9 commits intomainfrom
jc-add-api-object

Conversation

@josecolella
Copy link
Collaborator

@josecolella josecolella commented Nov 14, 2022

⚠️ Best reviewed commit-by-commit

This PR

  • This PR adds the API object that implements the specifications from api-initialization-and-configuration
  • This PR does not implement anything else, as I would like to break up the implementations into several PR for clearer intent

Related Issues

References #7

Notes

  • Add concurrent-ruby for thread safe data structures

Follow-up Tasks

  • Add client implementation

How to test

  1. Pull down branch
  2. Run bundle exec rspec

@josecolella josecolella changed the title [Spec Implementation] Add API object feat(spec): Add API implementation Nov 14, 2022
@josecolella josecolella force-pushed the jc-add-api-object branch 2 times, most recently from a7d73e8 to 7cc6102 Compare November 14, 2022 04:37
@josecolella josecolella force-pushed the jc-add-api-object branch 2 times, most recently from e2d7a44 to 61d0606 Compare November 14, 2022 05:59
Signed-off-by: Jose Colella <jose.colella@gusto.com>
Signed-off-by: Jose Colella <jose.colella@gusto.com>
Signed-off-by: Jose Colella <jose.colella@gusto.com>
Signed-off-by: Jose Colella <jose.colella@gusto.com>
- Remove Error class from SDK

Signed-off-by: Jose Colella <jose.colella@gusto.com>
Signed-off-by: Jose Colella <jose.colella@gusto.com>
Signed-off-by: Jose Colella <jose.colella@gusto.com>
Signed-off-by: Jose Colella <jose.colella@gusto.com>
Signed-off-by: Jose Colella <jose.colella@gusto.com>
Copy link
Member

@toddbaert toddbaert left a comment

Choose a reason for hiding this comment

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

This looks good to me! Thanks for keeping it digestible.

@josecolella
Copy link
Collaborator Author

Team going to merge this, but please let me know if there are any follow ups. I will continue to provide easy to review PR instead of everything in one PRs

@josecolella josecolella merged commit d6b0922 into main Nov 14, 2022
@josecolella josecolella deleted the jc-add-api-object branch November 14, 2022 18:18
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.

3 participants