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

Added a note that publishers created the accessibility metadata -95 #100

Merged
merged 2 commits into from
May 10, 2022

Conversation

GeorgeKerscher
Copy link
Collaborator

As you know I am new to git. If Charles or Gregorio could review and approve this and merge it, that would be great.

Copy link
Collaborator

@clapierre clapierre left a comment

Choose a reason for hiding this comment

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

Hi George,
In the line here I think there is a stray "a" included which should be removed.

perspective of a the supplier creating these files

@GeorgeKerscher
Copy link
Collaborator Author

GeorgeKerscher commented May 10, 2022 via email

@GeorgeKerscher
Copy link
Collaborator Author

Ha, I see. I’ll work on that. Best George From: Charles LaPierre @.> Sent: Tuesday, May 10, 2022 1:11 PM To: w3c/publ-a11y @.> Cc: George @.>; Author @.> Subject: Re: [w3c/publ-a11y] Added a note that publishers created the accessibility metadata -95 (PR #100) @clapierre requested changes on this pull request. Hi George, In the line here I think there is a stray "a" included which should be removed. perspective of a the supplier creating these files — Reply to this email directly, view it on GitHub <#100 (review)> , or unsubscribe https://github.com/notifications/unsubscribe-auth/ABW4OSFJXUDFNCDWSOUCZATVJKYF7ANCNFSM5VSRHXEA . You are receiving this because you authored the thread. https://github.com/notifications/beacon/ABW4OSFAKSATIEQS5M3GCODVJKYF7A5CNFSM5VSRHXEKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOHG3O6OA.gif Message ID: @.*** @.***> >

I mad the change, removing the "a" committed and pushed it. I don't think I need to issue a new PR.

@GeorgeKerscher GeorgeKerscher merged commit 52e8d80 into main May 10, 2022
@mattgarrish mattgarrish deleted the rachel95 branch July 11, 2023 16:52
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