Skip to content

Conversation

@twsouthwick
Copy link
Member

This change creates a module base class that enables exposing endpoints from the service in a standard way. This pattern is testable as well as ensures things like API key checks are done as expected.

This change creates a module base class that enables exposing endpoints from the service in a standard way. This pattern is somewhat testable as well as ensures things like API key checks are done as expected.
@twsouthwick twsouthwick force-pushed the tasou/common-remote-module branch from 9dcd4fd to 9c0968f Compare August 28, 2022 17:33
Copy link
Member

@mjrousos mjrousos left a comment

Choose a reason for hiding this comment

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

LGTM

@twsouthwick twsouthwick enabled auto-merge (squash) August 29, 2022 20:45
@twsouthwick twsouthwick merged commit bab1899 into main Aug 29, 2022
@twsouthwick twsouthwick deleted the tasou/common-remote-module branch August 29, 2022 23:40
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