Skip to content
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.

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

BP-167-Extra-Blocks-in-Editor #109

Merged
merged 14 commits into from Apr 21, 2024
Merged

BP-167-Extra-Blocks-in-Editor #109

merged 14 commits into from Apr 21, 2024

Conversation

gummyfrog
Copy link
Contributor

@gummyfrog gummyfrog commented Apr 20, 2024

Changes

Editor overhaul. List of changes:

  1. Styling. Editor looks closer to the home page.
  2. Error catching. Invalid workspaces are not saved. A workspace may be invalid for any of these reasons, checked in roughly this order:
    a. Missing Metadata Block
    b. Missing Metadata Fields
    c. Missing Game Loop Block
    d. Missing Exit Game Block
    e. Workspace does not compile
    f. Game does not run

Game running may not be something we end up checking at file save time. This can error-caught when running it from the home page if the background pygame run ends up being intrusive on the Pi. (Preliminary tests show that it is not.)

  1. Game icon saving.
  2. Overhaul the Python generator. No pygamezero.
  3. Overhaul available blocks. Should be much simpler now, reduced time to moving guy by quite a lot.
  4. Explicit working directories for the flask server, compiler, and compiled games. These should be safe to run from any directory now.
  5. Endpoints in Flask for running and compiling a game, either in test mode or actual run mode.
  6. Changed keypress handling

Tested & Verified

  1. Editor access from external machine
  2. Editor error catching, both on the blastpad and from external machine
  3. Running a game via endpoint
  4. Testing a game via endpoint
  5. App starts from start_x.sh
  6. Game runs from editor
  7. Game saves from editor

Tickets Covered

BP-218
[BP-167]

Copy link
Contributor

@tuk05348 tuk05348 left a comment

Choose a reason for hiding this comment

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

LGTM

@gummyfrog
Copy link
Contributor Author

LGTM

image

@gummyfrog gummyfrog merged commit a0f3830 into main Apr 21, 2024
3 checks passed
@Snarr Snarr deleted the BP-167-Extra-Blocks-in-Editor branch April 22, 2024 00:36
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.

None yet

3 participants