Skip to content
This repository has been archived by the owner on Jun 30, 2018. It is now read-only.

Issue 55 #102

Closed
wants to merge 2 commits into from
Closed

Issue 55 #102

wants to merge 2 commits into from

Conversation

inclusiveThinking
Copy link

Issue # 55 Clear Purpose

#36 Clear Controls from the COGA Task Force
#55 Clear Purpose - COGA Task Force
@awkawk
Copy link
Member

awkawk commented Feb 4, 2017

@inclusiveThinking this pull request is not just covering Issue 55 it has much more.

What needs to be in here is just what you have in lines 1638-1642, followed by a tag to close the section:

  •    <section class="sc">
    
  •    <h4>Clear Purpose</h4>
    
  •    <p class="conformance-level">A</p>
    
  •    <p class="change">New</p>
    
  •    <p>The main purpose of each page and section of content is clear. Extraneous information that is not directly relevant to the main purpose of a page is distinctly separated and programmatically determinable. For multi-step tasks, signposts should be provided to clarify the broader context including steps completed, current step and steps pending.</p>
    

The rest is either from a different SC, or is content that will belong in the understanding document.

It looks like you worked on issue 36 also and have included that in the same pull request, so clearly unintentional, but we need to have you remove the issue 36 content and the extra information from issue 55.

The location of these lines should also be between the original lines 743 and 744 - right after the "pronunciation" SC.
If you need help, please let me know.

@inclusiveThinking
Copy link
Author

Yep totally. I new something was wrong when I made the PR. I will resubmit. Just a heads up that I am submitting this and four other SC on behalf on other SC in the TF. After the PR is accepted they should be assigned to the appropriate SC Managers. I will add the SC Manger name in the PR description going forward.
Thanks!

@inclusiveThinking
Copy link
Author

I going to close and delete the branch

@inclusiveThinking inclusiveThinking deleted the ISSUE-55 branch February 4, 2017 18:18
@inclusiveThinking
Copy link
Author

Will open a clean PR

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants