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

Update index.md for #944 #1199

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Update index.md for #944 #1199

wants to merge 1 commit into from

Conversation

Alive24
Copy link

@Alive24 Alive24 commented Nov 17, 2024

What does this change?

Moves content from https://docs.codeyourfuture.io/archive/archived-pages/guides/common-responses/create-a-minimal-reproducible-example to https://curriculum.codeyourfuture.io/guides/getting-help/asking-questions/ after editing.

Common Content?

Not sure

Common Theme?

No

Issue number: #944

Org Content?

Block Type

Checklist

Who needs to know about this?

@illicitonion

Copy link

netlify bot commented Nov 17, 2024

Deploy Preview for cyf-curriculum canceled.

Name Link
🔨 Latest commit cde2098
🔍 Latest deploy log https://app.netlify.com/sites/cyf-curriculum/deploys/6739b942870e2b0008191b4c

Copy link

netlify bot commented Nov 17, 2024

👷 Deploy request for cyf-common pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit cde2098

Copy link

netlify bot commented Nov 17, 2024

👷 Deploy request for cyf-launch pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit cde2098

Copy link

netlify bot commented Nov 17, 2024

👷 Deploy request for cyf-piscine pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit cde2098

Copy link

netlify bot commented Nov 17, 2024

👷 Deploy request for cyf-tracks pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit cde2098

Copy link

netlify bot commented Nov 17, 2024

👷 Deploy request for cyf-sdc pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit cde2098

Copy link

netlify bot commented Nov 17, 2024

👷 Deploy request for cyf-itd pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit cde2098

@Alive24 Alive24 requested a review from illicitonion November 17, 2024 09:37
Copy link

netlify bot commented Nov 19, 2024

Deploy Preview for cyf-programming ready!

Name Link
🔨 Latest commit cde2098
🔍 Latest deploy log https://app.netlify.com/sites/cyf-programming/deploys/6739b94214493800087d074e
😎 Deploy Preview https://deploy-preview-1199--cyf-programming.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 99 (🟢 up 14 from production)
Accessibility: 100 (no change from production)
Best Practices: 92 (no change from production)
SEO: 90 (no change from production)
PWA: -
View the detailed breakdown and full score reports

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

@illicitonion
Copy link
Member

Thanks for picking this up!

I think one of the ways the existing content is used is to be directly linked to (e.g. in https://codeyourfuture.slack.com/archives/C59R2722F/p1625229921143000?thread_ts=1625229023.140700&cid=C59R2722F), and that's hard to do with a tooltip.

Could you pull out a new page (e.g. org-cyf-guides/content/getting-help/minimal-reproducible-examples/index.md) and put the content in there (and link to it where you currently have a tooltip)?

Sorry for not mentioning this requirement on the original ticket!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 📋 Backlog
Development

Successfully merging this pull request may close these issues.

2 participants