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

Move kiosk game list to targetconfig.json #6122

Merged
merged 4 commits into from
Oct 17, 2023

Conversation

eanders-ms
Copy link
Contributor

@eanders-ms eanders-ms commented Oct 16, 2023

Companion PR in pxt: microsoft/pxt#9723

"imageUrl": "/static/tutorials/pizza-party/pizza-party-img.png",
"largeImageUrl": "/static/tutorials/pizza-party/tmnt.gif"
"imageUrl": "/static/tutorials/pizza-party/playtime.png",
"largeImageUrl": "/static/tutorials/pizza-party/playtime.png"
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this just an automated change?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes. Sometimes they just happen. I'm not sure what triggers them.

Copy link
Member

Choose a reason for hiding this comment

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

iirc this file is generated on pxt serve going through and grabbing the first thing in each row, so will update whenever that changes

@@ -1,4 +1,5 @@
{
"TODO: Delete this file after the next Arcade release": 1,
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm guessing you can't delete this now because it's still being used?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Changes to the docs folder take affect almost immediately, and that can impact the current release. The released version of kiosk doesn't reference this file, so it would be fine. The beta version does though. Not the end of the world. Bumping beta would fix that.

Copy link
Contributor

@kimprice kimprice left a comment

Choose a reason for hiding this comment

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

Just a couple questions for my own understanding.

"imageUrl": "/static/tutorials/pizza-party/pizza-party-img.png",
"largeImageUrl": "/static/tutorials/pizza-party/tmnt.gif"
"imageUrl": "/static/tutorials/pizza-party/playtime.png",
"largeImageUrl": "/static/tutorials/pizza-party/playtime.png"
Copy link
Member

Choose a reason for hiding this comment

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

iirc this file is generated on pxt serve going through and grabbing the first thing in each row, so will update whenever that changes

Comment on lines 375 to 378
"id": "_Y8VUmcHYsdsy",
"name": "GalgaMulti",
"description": "Controlling a starship, both players will destroy the Galga forces, while avoiding enemies. Each player has 3 lives, game ends once a player runs out of lives.",
"highScoreMode": "SingleAscending"
Copy link
Member

Choose a reason for hiding this comment

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

nit: prefer indenting but that's only for readability / doesn't really matter if you don't feel like doing it

@eanders-ms eanders-ms merged commit b8e27a1 into master Oct 17, 2023
3 checks passed
@eanders-ms eanders-ms deleted the eanders-ms/kiosk-targetconfig branch October 17, 2023 03:04
eanders-ms added a commit that referenced this pull request Oct 24, 2023
* Move kiosk game list to targetconfig.json

* fix indenting
eanders-ms added a commit that referenced this pull request Oct 24, 2023
* Add kiosk-data docs folder (#6009)

* Remove /kiosk site, redirect to /--kiosk. (#6030)

* Remove built kiosk from docs

* Add redirect for kiosk

* Remove kiosk.html

* Remove old kiosk sources (#6036)

* Move kiosk sfx to arcade repo (#6068)

* Kiosk: Run high pass filter on UI audio (#6081)

* Move kiosk game list to targetconfig.json (#6122)

* Move kiosk game list to targetconfig.json

* fix indenting

* Bump pxt-arcade-sim -> v0.11.9

* bump pxt-core -> 8.5.54

* projects.md

* Revert "Remove kiosk.html"

This reverts commit 0b1189c.

* Revert "Remove built kiosk from docs"

This reverts commit 5d52d2b.

* bump pxt-core -> 8.5.55
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.

3 participants