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

Conduct WCAG pre-assessment for Share Widget (GCWeb wet-368) #2197

Merged
merged 10 commits into from
Sep 7, 2023

Conversation

hongbinyu413
Copy link
Contributor

No description provided.

@hongbinyu413 hongbinyu413 temporarily deployed to github-ci July 12, 2023 18:43 — with GitHub Actions Inactive
@hongbinyu413 hongbinyu413 temporarily deployed to github-ci July 17, 2023 13:06 — with GitHub Actions Inactive
Copy link
Member

@duboisp duboisp left a comment

Choose a reason for hiding this comment

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

Rename the folder "share widget" for "share".
So all those files are going to be in the folder: common/share/reports

If you did an assessment and not a pre-assessment, please remove the word "Partielle" and "Partial. " inside the page description

And there is a few change in the HTML page front-matter

common/share widget/reports/ally-1-en.html Outdated Show resolved Hide resolved
common/share widget/reports/ally-1-fr.html Outdated Show resolved Hide resolved
common/share widget/reports/ally-1-fr.html Outdated Show resolved Hide resolved
common/share widget/reports/ally-1-en.html Outdated Show resolved Hide resolved
common/share widget/reports/pre-ally-1.json Outdated Show resolved Hide resolved
@hongbinyu413 hongbinyu413 temporarily deployed to github-ci August 31, 2023 13:24 — with GitHub Actions Inactive
common/share widget/reports/ally-1-en.html Outdated Show resolved Hide resolved
common/share widget/reports/ally-1-fr.html Outdated Show resolved Hide resolved
common/share widget/reports/pre-ally-1.json Outdated Show resolved Hide resolved
@hongbinyu413 hongbinyu413 temporarily deployed to github-ci August 31, 2023 15:40 — with GitHub Actions Inactive
duboisp
duboisp previously approved these changes Sep 6, 2023
@duboisp duboisp temporarily deployed to github-ci September 6, 2023 21:25 — with GitHub Actions Inactive
@duboisp
Copy link
Member

duboisp commented Sep 6, 2023

I added the index.json-ld file for the share widget. I am going to squash and merge once the CI script is successful.

This PR is only a content change.

@duboisp duboisp merged commit 7814a3c into wet-boew:master Sep 7, 2023
1 check passed
SebastianBurke pushed a commit to SebastianBurke/GCWeb that referenced this pull request Mar 6, 2024
* Conduct WCAG pre-assessment for Share Widget (GCWeb wet-368)
* wet-368 Conduct WCAG pre-assessment for Share Widget (GCWeb)
* Update common/share widget/reports/ally-1-en.html
* Update common/share widget/reports/ally-1-fr.html
* Update common/share widget/reports/ally-1-fr.html
* Update common/share widget/reports/ally-1-en.html
* Update common/share widget/reports/pre-ally-1.json
* wet-368 Conduct WCAG pre-assessment for Share Widget (GCWeb)
* Share widget - Add index.json-ld file for the component description

---------

Co-authored-by: hongbinyu413 <[email protected]>
Co-authored-by: Pierre Dubois <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants