Skip to content

Begin uploading results to matched_scrambles#13934

Open
gregorbg wants to merge 16 commits intothewca:mainfrom
gregorbg:feature/upload-to-matched-scrambles
Open

Begin uploading results to matched_scrambles#13934
gregorbg wants to merge 16 commits intothewca:mainfrom
gregorbg:feature/upload-to-matched-scrambles

Conversation

@gregorbg
Copy link
Copy Markdown
Member

@gregorbg gregorbg commented Apr 6, 2026

This PR eases in the migration path towards the integrated Scrambles Matcher.

We currently hard-upload every scramble to the productive, public Scrambles table directly, right at the moment that the Delegate uploads the results JSON. This is really as awkward as it sounds.

With the work on integrating scrambles matcher, we now suddenly have matchup tables that can serve as a suitable replacement for temporary scramble posting, kinda like a smarter version of inbox_scrambles.

However, in order to not disturb existing results posting procedures, we need to smoothly "ease in" the new data flow. So this PR does the following:

  • When a Results JSON is uploaded, it fully follows the "old" way of doing things (ie hard-uploads to scrambles directly) AND it also uploads to matched_scrambles in the background.
  • The data in matched_scrambles is not used productively (yet), it's just silently being written
  • At some point in the near future (sponaneous guess: 3-4 days from this PR) we can merge in another PR that actually arms the validators to read from matched_scrambles, and also stops uploading to the public scrambles table upon upload.

This procedure is mostly necessary in order to not disturb any ongoing submissions that are either pending or in the process of being posted while this PR is being deployed.

@gregorbg gregorbg marked this pull request as ready for review April 6, 2026 09:31
@gregorbg gregorbg force-pushed the feature/upload-to-matched-scrambles branch from 48a330f to 9f8b6b7 Compare April 6, 2026 21:25
@gregorbg gregorbg force-pushed the feature/upload-to-matched-scrambles branch from 9f8b6b7 to 9c6ac3e Compare April 7, 2026 04:42
Copy link
Copy Markdown
Member

@danieljames-dj danieljames-dj left a comment

Choose a reason for hiding this comment

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

Awesome!

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