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

Changes corresponding to an implementation of a core api. #154

Draft
wants to merge 5 commits into
base: next
Choose a base branch
from

Conversation

albu-diku
Copy link

@albu-diku albu-diku commented Nov 25, 2024

No description provided.

@albu-diku albu-diku force-pushed the addition/core-api branch 2 times, most recently from 267875b to 067b2c2 Compare November 27, 2024 13:16
@albu-diku albu-diku force-pushed the addition/core-api branch 3 times, most recently from 85f9adb to 2eb2c72 Compare December 23, 2024 16:10
@albu-diku albu-diku changed the base branch from edge to next January 27, 2025 12:37
@albu-diku albu-diku force-pushed the addition/core-api branch 4 times, most recently from 42cc644 to a9bea6f Compare January 28, 2025 09:03
@albu-diku albu-diku marked this pull request as ready for review January 28, 2025 09:09
@albu-diku albu-diku force-pushed the addition/core-api branch 4 times, most recently from 32c2b91 to 9875abb Compare February 5, 2025 14:43
@albu-diku albu-diku marked this pull request as draft February 25, 2025 09:26
Add the logic to automatically package values as a bundle.

Implement declarative argument definitions and logic to bundle values.

carve out payloads and rework them to make use of the validation helper

Expand the arguments module with the notions of a defined grouping of arguments
(ArgumentBundleDefinition) and bundles of particular arguments (ArgumentBundle).

Use this as the mechanism by which payloads are checked for validity. As part
of declaring a bundle definition the expected positional arguments are declared
so implement basic length checks that catch missing positional arguments which
were required. Each argument itself is also tested aganst an optional validity
function which can be specified at the point of definition.

try to combine payloads with the argument stuff

move the test

further re-integration

match the other branch

fixup

fixup

fixup

repair and test error conditions

fixup

fixup

fixup

quack like a named tuple

raise uniform payload exceptions and properly handle dictionaries
Do the core work necessary to have a first response served to a GET
request and some basic wiring to allow submission of JSON via POST.

CHECKPOINT: running coreapi server

overrides? not yet sure why this was needed

Implement a user creation endpoint in the server.

Back a POST handler onto the refactored createuser functionality file which
exposes a function that can be invoked programatically with a configuration.

Split out the response data decoding chunk.

axe depedency pn reworkings on userapi

fixups - move closer to the server working without createuser changes

Implement finding users by email address.

relocate

fixup

carve out payloads and rework them to make use of the validation helper

shut up flake complaints about things that have changed

further fixup

another
@albu-diku albu-diku force-pushed the addition/core-api branch from 08dd5ec to f093e2f Compare March 4, 2025 09:23
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