Skip to content

AAI-236 Bugfix for datetimes in JSON#24

Merged
marius-mather merged 3 commits into
mainfrom
bugfix-json-datetimes
Jun 5, 2025
Merged

AAI-236 Bugfix for datetimes in JSON#24
marius-mather merged 3 commits into
mainfrom
bugfix-json-datetimes

Conversation

@marius-mather

@marius-mather marius-mather commented Jun 5, 2025

Copy link
Copy Markdown
Collaborator

Description

AAI-236: currently getting an error "datetime is not JSON-serializable" when trying to register on Galaxy. Caused by not converting to JSON-friendly types before putting things in POST requests etc.

Changes

  • Make sure we use model_dump(mode='json') before sending POST data to Auth0
  • Tests for handling of types like datetime

Checklist

  • I have commented my code, particularly in hard-to-understand areas
  • I have added unit / integration tests that prove my fix is effective or that my feature works
  • I have run all tests locally and they pass
  • I have updated the documentation (if applicable)

How to Test Manually (if necessary)

Run uv run pytest

@marius-mather marius-mather merged commit 0e28d34 into main Jun 5, 2025
1 check passed
@amandazhuyilan amandazhuyilan deleted the bugfix-json-datetimes branch June 19, 2026 03:21
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