Skip to content

Initial implementation of simple data consent api. - #3977

Merged
dsubak merged 2 commits into
mainfrom
dansubak/202609_add_data_consent_api
Sep 24, 2026
Merged

dsubak merged 2 commits into
mainfrom
dansubak/202609_add_data_consent_api

Conversation

@dsubak

@dsubak dsubak commented Sep 16, 2026 •

Copy link
Copy Markdown
Contributor

What are the relevant tickets?

https://github.com/mitodl/hq/issues/13277

Description (What does it do?)

Adds the simplest possible API for tracking user data sharing consent on a per-contract basis. It is simply an API which takes a consent value for a given contract and records that for the authenticated user. We may need more complicated logic at some point, but that remains to be seen at present.

How can this be tested?

@dsubak
dsubak added this pull request to stack #3970 September 16, 2026 15:38
@github-actions

Copy link
Copy Markdown

OpenAPI Changes

Show/hide changes
## Changes for v0.yaml:
1 changes: 0 error, 0 warning, 1 info
info	[endpoint-added] at head/openapi/specs/v0.yaml
	in API POST /api/v0/b2b/data_consent/{contract_id}/
		endpoint added



## Changes for v1.yaml:
1 changes: 0 error, 0 warning, 1 info
info	[endpoint-added] at head/openapi/specs/v1.yaml
	in API POST /api/v0/b2b/data_consent/{contract_id}/
		endpoint added



## Changes for v2.yaml:
1 changes: 0 error, 0 warning, 1 info
info	[endpoint-added] at head/openapi/specs/v2.yaml
	in API POST /api/v0/b2b/data_consent/{contract_id}/
		endpoint added



Unexpected changes? Ensure your branch is up-to-date with main (consider rebasing).

@dsubak
dsubak marked this pull request as ready for review September 16, 2026 17:59
@dsubak
dsubak force-pushed the dansubak/202609_add_data_consent_api branch from bb768c9 to befe9fd Compare September 17, 2026 18:45
@dsubak
dsubak force-pushed the dansubak/202609_add_data_consent_api branch from befe9fd to 2ec8728 Compare September 17, 2026 20:44
@dsubak
dsubak force-pushed the dansubak/202609_add_data_consent_api branch 6 times, most recently from 641548a to 90dcb09 Compare September 24, 2026 12:28
Base automatically changed from dansubak/202609_add_data_consent_fields_to_through_model_pt_2 to main September 24, 2026 15:33
@dsubak
dsubak force-pushed the dansubak/202609_add_data_consent_api branch from 90dcb09 to ec3f859 Compare September 24, 2026 15:33
@dsubak
dsubak force-pushed the dansubak/202609_add_data_consent_api branch from ec3f859 to 4ece6f6 Compare September 24, 2026 18:16

@jkachel jkachel left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM 👍

@dsubak
dsubak merged commit 78fa510 into main Sep 24, 2026
14 checks passed
@dsubak
dsubak deleted the dansubak/202609_add_data_consent_api branch September 24, 2026 19:01
This was referenced Sep 28, 2026
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.

2 participants