Skip to content

Cromper mkst #1690

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 20 commits into
base: cromper-2
Choose a base branch
from
Open

Cromper mkst #1690

wants to merge 20 commits into from

Conversation

mkst
Copy link
Collaborator

@mkst mkst commented Sep 29, 2025 •
edited
Loading

removing the /platform route from urls has broken some stuff. All SSR that does a get("/platform") no longer works.

I feel like we should route these requests directly to cromper rather than have a route only for SSR?

I've added a temporary fix to route anything /platform related to cromper (or /api when coming from the client)

mkst and others added 10 commits September 25, 2025 19:05
@mkst
Fix diff cell background color setting (#1681)
d10d6af
@mkst
Remove 'include_presets' from /api/platform endpoint (#1683)
80fbb78 
* Remove 'include_presets' from /api/platform endpoint

* 🐶 ruff ruff

* Remove broken test
@mkst
Fix 500 when creating scratches with a preset that have libraries (#1685
1b7fe4d 
)

* Fix 500 when creating scratches with a preset that have libraries

* Allow users to specify libraries when creating scratches

* Add validation to user-supplied libraries

* 🐕

* mypy, yourpy, ourpy

* woof
@mkst
Remove plausible (#1686)
560888d
@mkst
Remove 'num_scratches' from platform endpoint (#1688)
1f0dc00
@mkst
Merge branch 'main' of github.com:decompme/decomp.me into cromper-mkst
27255c6
@mkst
Split handlers out of main.py
34483f4
@mkst
Move wrappers into wrappers/ dir
b4ac22d
@mkst
Rudimentary 'Library' support in cromper
436cd9c
@mkst
ruff
eb517d4
@mkst mkst requested a review from bates64 as a code owner September 29, 2025 18:09
ethteck
ethteck reviewed Sep 30, 2025
View reviewed changes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ethteck ethteck ethteck left review comments

@bates64 bates64 Awaiting requested review from bates64 bates64 is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mkst @ethteck