Skip to content

[PXCT-160] Appwizard tutorial #2187

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 22 commits into from
Mar 25, 2025
Merged

Conversation

BenjaminDannegard
Copy link
Contributor

@BenjaminDannegard BenjaminDannegard commented Sep 24, 2024

What This PR Changes

  • Added a tutorial for using AppWizard on the Giga Display Shield

Contribution Guidelines

Copy link

github-actions bot commented Sep 24, 2024

Preview Deployment

🚀 Preview this PR: https://67e2d83a3336b46f5d96bb47--docs-content.netlify.app
📍 Commit SHA: b04d8f7

BenjaminDannegard and others added 7 commits October 17, 2024 13:06
…/13.appwizard-guide/content.md

Co-authored-by: Leonardo Cavagnis <[email protected]>
…/13.appwizard-guide/content.md

Co-authored-by: Leonardo Cavagnis <[email protected]>
…/13.appwizard-guide/content.md

Co-authored-by: Leonardo Cavagnis <[email protected]>
…/13.appwizard-guide/content.md

Co-authored-by: Leonardo Cavagnis <[email protected]>
…/13.appwizard-guide/content.md

Co-authored-by: Leonardo Cavagnis <[email protected]>
@BenjaminDannegard BenjaminDannegard changed the title Benjamindannegard/appwizard tutorial [PXCT-160] Appwizard tutorial Mar 14, 2025
Copy link
Collaborator

@mcmchris mcmchris left a comment

Choose a reason for hiding this comment

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

Hey @BenjaminDannegard I reviewed the tutorial and left some minor comments.

I would recommend for future tutorials to set your computer background to a solid color, (hex: D6EEF0) and to take screenshots directly on it so to avoid cropping images and giving consistency to the style.

BenjaminDannegard and others added 7 commits March 14, 2025 17:50
…/13.appwizard-guide/content.md

Co-authored-by: Christopher Méndez <[email protected]>
…/13.appwizard-guide/content.md

Co-authored-by: Christopher Méndez <[email protected]>
…/13.appwizard-guide/content.md

Co-authored-by: Christopher Méndez <[email protected]>
…/13.appwizard-guide/content.md

Co-authored-by: Christopher Méndez <[email protected]>
…/13.appwizard-guide/content.md

Co-authored-by: Christopher Méndez <[email protected]>
Copy link
Collaborator

@mcmchris mcmchris left a comment

Choose a reason for hiding this comment

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

LGTM! Now

@jcarolinares jcarolinares requested review from leonardocavagnis and removed request for karlsoderby and leonardocavagnis March 25, 2025 16:07
@jcarolinares jcarolinares dismissed leonardocavagnis’s stale review March 25, 2025 16:08

Changes and feedback were applied, dismissing review just to unblick the merge without bothering people

Copy link
Contributor

@jcarolinares jcarolinares left a comment

Choose a reason for hiding this comment

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

LGTM with the latest changes

@mcmchris mcmchris merged commit 9e127ed into main Mar 25, 2025
4 checks passed
@mcmchris mcmchris deleted the benjamindannegard/appwizard-tutorial branch March 25, 2025 16:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants