Skip to content

Showcase Submission for processing #1651

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

Merged
merged 2 commits into from
Jul 16, 2024
Merged

Conversation

ch00ooch00oo
Copy link
Collaborator

@ch00ooch00oo ch00ooch00oo commented Jul 9, 2024

Thank you EclipseMaster0 for your contribution! A member of the Coding Train team will review it shortly.

image

@ch00ooch00oo ch00ooch00oo added the showcase Passenger Showcase Submission label Jul 9, 2024
Copy link

netlify bot commented Jul 9, 2024

Deploy Preview for codingtrain ready!

Name Link
🔨 Latest commit cb40521
🔍 Latest deploy log https://app.netlify.com/sites/codingtrain/deploys/669516af6aecc10008a54b57
😎 Deploy Preview https://deploy-preview-1651--codingtrain.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@shiffman
Copy link
Member

Hi @devksankar, thanks for submitting! I prefer for the images to be a screenshot of the visual output of the code itself, would you like to update this?

@fturmel
Copy link
Collaborator

fturmel commented Jul 15, 2024

I updated the thumbnail for the submission.

@devksankar Your Blobs class should have the extension .pde to work properly.

CleanShot 2024-07-15 at 08 23 26@2x

@shiffman
Copy link
Member

Thank you @fturmel!

@shiffman shiffman merged commit 29242aa into main Jul 16, 2024
5 checks passed
@shiffman shiffman deleted the showcase-eclipsemaster0-1720547071 branch July 16, 2024 07:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting update showcase Passenger Showcase Submission
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants