Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Refactor backend-specific clients to use a new Backend interface #515

Closed
wants to merge 1 commit into from

Conversation

gadomski
Copy link
Member

Related Issue(s):

Description:

This is a work-in-progress PR to demonstrate an alternative approach to decoupling backends from the API layer. Two main features:

  • A new Backend interface that does the actual fetching from the database/data store
  • A new PaginationLinks structure that allows a backend to provide "unresolved" pagination links, that are resolved at the client layer

PR Checklist:

  • Code is formatted and linted (run pre-commit run --all-files)
  • Tests pass (run make test)
  • Documentation has been updated to reflect changes, if applicable, and docs build successfully (run make docs)
  • Changes are added to the CHANGELOG.

@gadomski
Copy link
Member Author

Closing as OBE, but I'm going to keep the branch around as a PoC for a while.

@gadomski gadomski closed this Feb 22, 2023
@gadomski gadomski deleted the backends branch March 9, 2023 14:58
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant