Skip to content

Use Avaje Jsonb for Javalin JSON#2512

Merged
mcm001 merged 2 commits into
PhotonVision:mainfrom
thatcomputerguy0101:javalin-avaje
Jun 10, 2026
Merged

Use Avaje Jsonb for Javalin JSON#2512
mcm001 merged 2 commits into
PhotonVision:mainfrom
thatcomputerguy0101:javalin-avaje

Conversation

@thatcomputerguy0101

Copy link
Copy Markdown
Contributor

Description

This updates to the release version of Avaje Jsonb and switches Javalin to use Avaje Jsonb as its backend to fix some serialization bugs. No tests are written since no tests are set up for photon-server, but there probably should be to catch problems like this.

Meta

Merge checklist:

  • Pull Request title is short, imperative summary of proposed changes
  • The description documents the what and why, including events that led to this PR
  • If this PR changes behavior or adds a feature, user documentation is updated
  • If this PR touches photon-serde, all messages have been regenerated and hashes have not changed unexpectedly
  • If this PR touches configuration, this is backwards compatible with all settings going back to the previous seasons's last release (seasons end after champs ends)
  • If this PR touches pipeline settings or anything related to data exchange, the frontend typing is updated
  • If this PR addresses a bug, a regression test for it is added
  • If this PR adds a dependency, the license has been checked for compatibility and steps taken to follow it

@thatcomputerguy0101 thatcomputerguy0101 requested a review from a team as a code owner June 9, 2026 22:05
@github-actions github-actions Bot added the backend Things relating to photon-core and photon-server label Jun 9, 2026
mcm001
mcm001 previously approved these changes Jun 9, 2026

@mcm001 mcm001 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.

Let's definitely scope tests for future work, focusing on high reward/low effort options

@mcm001 mcm001 enabled auto-merge (squash) June 9, 2026 22:32
auto-merge was automatically disabled June 9, 2026 23:17

Head branch was pushed to by a user without write access

@mcm001 mcm001 enabled auto-merge (squash) June 10, 2026 00:19
@mcm001 mcm001 merged commit c04c8d7 into PhotonVision:main Jun 10, 2026
56 of 57 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend Things relating to photon-core and photon-server

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants