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

Completes initial design and refactor of dynamic configuration #17

Merged
merged 8 commits into from
Dec 3, 2023

Conversation

devraj
Copy link
Member

@devraj devraj commented Dec 3, 2023

This PR implements the majority of #5 still pending from the work is:

  • Caching discovery results to improve performance
  • Timestamps on all API calls to keep a cache date

The remainder of the work to be conducted in the dto-implementation branch

this a commit in a separate branch for me to experinment implementing
dynamic discovery of the api endpoints, at the moment because of
import cycles the singleton seems to be overrwritten

REFS #5
we have the discovery object being initialised when the base class
initialises and then the discovery fetches the url, note that
the href that the discover will return are fully qualified addresses

this drops using the api_base prefix

post this we need to formalies the configuration and syntax so this
scales to all endpoints

we are also yet to test for unlicensed features
REFS #5
adds context around the design of the classes and lifecycle for dynamic
discovery of api endpoints

REFS #5
@devraj devraj merged commit 163a2c0 into dto-implementation Dec 3, 2023
@devraj devraj deleted the dynamic-config branch December 18, 2023 05:32
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.

1 participant