Skip to content

update instructions of dom merge conflict workshop - #1991

Open
Poonam-raj wants to merge 1 commit into
mainfrom
itp-dom-merge-conflict-fix
Open

update instructions of dom merge conflict workshop#1991
Poonam-raj wants to merge 1 commit into
mainfrom
itp-dom-merge-conflict-fix

Conversation

@Poonam-raj

Copy link
Copy Markdown
Contributor

to direct people more directly on the DOM task, removing every task except buttons and counter, to get people to the point of resolving a merge conflict

to direct people more directly on the DOM task, removing every task except buttons and counter, to get people to the point of resolving a merge conflict
@netlify

netlify Bot commented Aug 24, 2026

Copy link
Copy Markdown

Deploy Preview for cyf-common ready!

Name Link
🔨 Latest commit dace213
🔍 Latest deploy log https://app.netlify.com/projects/cyf-common/deploys/6a8c5c38bcd8a50008b1ea08
😎 Deploy Preview https://deploy-preview-1991--cyf-common.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

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

@netlify

netlify Bot commented Aug 24, 2026

Copy link
Copy Markdown

Deploy Preview for cyf-curriculum ready!

Name Link
🔨 Latest commit dace213
🔍 Latest deploy log https://app.netlify.com/projects/cyf-curriculum/deploys/6a8c5c389b89b60007824b3a
😎 Deploy Preview https://deploy-preview-1991--cyf-curriculum.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 2 from production)
Accessibility: 100 (no change from production)
Best Practices: 92 (no change from production)
SEO: 100 (no change from production)
PWA: -
View the detailed breakdown and full score reports
🤖 Make changes Run an agent on this branch

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

@illicitonion illicitonion left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM!

Comment on lines 45 to 48
```
git switch main
git pull origin main
```

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
```
git switch main
git pull origin main
```
```
git switch main
git pull origin main
```
Then each create your own branch with a unique name.

@github-project-automation github-project-automation Bot moved this from 📋 Backlog to 👀 In review in Curriculum Planning Aug 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 👀 In review

Development

Successfully merging this pull request may close these issues.

2 participants