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

Adding XboxPlaydates quote app after tests #1751

Merged
merged 4 commits into from
Sep 5, 2023

Conversation

PalmettoBling
Copy link
Contributor

@PalmettoBling PalmettoBling commented Aug 18, 2023

Description

Adding in the XboxPlaydates app. The app displays a random quote pulled via the Xbox Playdates API from the Xbox Playdates quote database, which is made up of community submitted quotes of things said during our community streams. The App displays the quote number that can be referenced, the quote itself and who said the quote.

xboxplaydates

Copilot

🤖 Generated by Copilot at 0a304b6

Summary

🎮📝🧠

This pull request adds a new applet called XboxPlaydates, which shows a random quote from the Xbox Playdates website on the Tidbyt device. It includes a manifest file and a Starlark code file for the applet.

XboxPlaydates app
Starlark code and manifest
Winter fun on Tidbyt

Walkthrough

  • Create and populate a manifest file for the XboxPlaydates applet (link)
  • Implement the applet logic and UI in Starlark (link)

@tidbyt
Copy link

tidbyt bot commented Aug 18, 2023

⚠️ The automated review process is experimental and likely has bugs. Please bear with us as we iron out the kinks and enable you to ship changes at high velocity 🚀

All Set 🎉

This change is ready to go! To merge your change, simply comment with a :shipit: (:shipit:) emoji on this pull request and our robots will take it from there.

Manual Review Required

Hang tight! A Tidbyt engineer will be by shortly to review your change. Here is what they will be looking for:

Test Details
App Dir All files are in a single app directory
🟡 Modules Usage of http.star requires review
Original Author The original author matches the PR author

Previews

apps/xboxplaydates/xboxplaydates.star:

@tidbyt-bot
Copy link

tidbyt-bot commented Aug 18, 2023

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@PalmettoBling
Copy link
Contributor Author

I have read the CLA Document and I hereby sign the CLA

@PalmettoBling
Copy link
Contributor Author

For the record of using the http.star:
image
The Xbox Playdates API is a Node / Express website/Twitch bot I have running in Azure with the documents being retrieved stored in an Azure Cosmos Database. I utilize the app and API regularly for other functions. The API is used for a Google Chrome / MS Edge extension, discord bots, website usage, Twitch chatbots, ect. and is resilient.

apps/xboxplaydates/xboxplaydates.star Outdated Show resolved Hide resolved
Copy link
Contributor Author

@PalmettoBling PalmettoBling left a comment

Choose a reason for hiding this comment

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

Updated file to remove function and library call. Committed to branch.

apps/xboxplaydates/xboxplaydates.star Outdated Show resolved Hide resolved
@PalmettoBling PalmettoBling requested a review from matslina August 31, 2023 12:50
@matslina matslina merged commit 8e2d560 into tidbyt:main Sep 5, 2023
@tidbyt tidbyt locked as resolved and limited conversation to collaborators Sep 8, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants