- From: Léonie Watson <tink@tink.uk>
- Date: Fri, 10 Feb 2017 10:35:55 +0000
- To: Andrew Kirkpatrick <akirkpat@adobe.com>, "lisa.seeman" <lisa.seeman@zoho.com>
- Cc: public-cognitive-a11y-tf <public-cognitive-a11y-tf@w3.org>, Joshue O Connor <josh@interaccess.ie>, "W3c-Wai-Gl-Request@W3. Org" <w3c-wai-gl@w3.org>
This may not work for WCAG, but in case it helps.. A project I work on has a Github issue for each thing to be reviewed. The content to be reviewed is the first comment in the issue. Whenever the draft is updated (based on Github comments or feedback from other sources), the first comment in the thread is edited to reflect the changed content. Providing the SC manager is able to edit the issue thread, they would be the ones responsible for updating the draft SC in response to feedback from different places. It does mean that one person has to be responsible for updating the draft, but the advantage is that all comments are kept in the one thread, instead of split over the original issue and multiple subsequent PR threads. It also means there is only ever one definitive place where the content to be reviewed can be found. Léonie -- @LeonieWatson tink.uk Carpe diem On 09/02/2017 23:00, Andrew Kirkpatrick wrote: > We are also worried about tracking all the comments so people can find > them. Please feel free to advice on that too! > > This is one of the advantages of not opening a second pull request – the > comments stay with the pull request. There is a transition when the pull > request is made in that the comments initially were in the issue, but > once the pull request is opened: > > 1) the issue should be closed, and a link to the pull request added to > the issue > 2) Comments should be made on the pull request and not on the issue. > > Thanks, > AWK > > > ---- On Thu, 09 Feb 2017 21:29:10 +0200 *Andrew > Kirkpatrick<akirkpat@adobe.com <mailto:akirkpat@adobe.com>>* wrote ---- > > Lisa, > I’m traveling today and it is late at night for Josh, so we may not > be able to answer today, so do not assume that the process you > describe below is correct if you don’t hear back today, it may be > tomorrow before I can respond fully. > > Thanks, > AWK > > Andrew Kirkpatrick > Group Product Manager, Standards and Accessibility > Adobe > > akirkpat@adobe.com <mailto:akirkpat@adobe.com> > http://twitter.com/awkawk > > From: "lisa.seeman@zoho.com <mailto:lisa.seeman@zoho.com>" > <lisa.seeman@zoho.com <mailto:lisa.seeman@zoho.com>> > Date: Thursday, February 9, 2017 at 10:43 > To: "public-cognitive-a11y-tf@w3.org > <mailto:public-cognitive-a11y-tf@w3.org>" > <public-cognitive-a11y-tf@w3.org > <mailto:public-cognitive-a11y-tf@w3.org>> > Cc: Andrew Kirkpatrick <akirkpat@adobe.com > <mailto:akirkpat@adobe.com>>, Joshue O Connor <josh@interaccess.ie > <mailto:josh@interaccess.ie>>, WCAG <w3c-wai-gl@w3.org > <mailto:w3c-wai-gl@w3.org>> > Subject: process for second pull request > > For doing a second pull request I suggested to coga the following > process (because none of us knew what to do) > > Please *let us know today* if anything here is incorrect. > (changes are in bold) > > > # Develop a pull request on GitHub containing the new wording. > > 1. Fork the WCAG21 repository to your own GitHub account > 2. Create a branch off of the Master branch (use a logical name for > the branch, e.g. "Accessible authentication version 2 ”) > 3. Open the directory "guidelines" in your new branch > 4. open index. html , press the edit box > 5. navigate to the right guideline and issert the following with > the SC text in the following mark up > > <section class="sc"> > > <h4> </h4> > > <p class="conformance-level">AA</p> > > <p></p> > > </section> > > 1. Navigate to the glossary and add any new glossary items > (hopefully in the right place) > 2. Save and create the pull request for the Master branch. > > # When the Pull request is made: > > 1. *Provide the link to the first pull request with some text > along the lines of "this is a second pull request in response to > the comments from the pull request at ...)* > 2. *Provide the link to the original issue in the pull request > comments as well.* > 3. * Provide the link to the second pull request from the first > pull requet* > > ll the best > > Lisa Seeman > > LinkedIn <http://il.linkedin.com/in/lisaseeman/>, Twitter > <https://twitter.com/SeemanLisa> > > > All the best > > Lisa Seeman > > LinkedIn <http://il.linkedin.com/in/lisaseeman/>, Twitter > <https://twitter.com/SeemanLisa> > > > >
Received on Friday, 10 February 2017 10:36:31 UTC