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

Remove WPS references from all documentation #820

Closed
brianhatchl opened this issue Jun 10, 2016 · 6 comments
Closed

Remove WPS references from all documentation #820

brianhatchl opened this issue Jun 10, 2016 · 6 comments

Comments

@brianhatchl
Copy link
Contributor

refs #800 and #680

@mikejeffe
Copy link

mikejeffe commented Jun 10, 2016

The Hootenanny Developer doc is the biggest offender with some arch. diagrams and a few explicit references to WPS. The other mention is in the User Guide. Shouldn't be too much work to remove these assuming we are all fine with modifying the architecture images in the developer guide. I'll take point on this and merge into develop.

@mikejeffe
Copy link

mikejeffe commented Jun 13, 2016

5ee7fe1
@sisskind or @bwitham can either of you review the doc changes to Hootenanny Developer and User Guide? Most changes are in developer guide where I re-worked the arch diagram to remove WPS references and removed it from text. There was only one instance in User Guide which I removed. Branch is 820docs. Thanks

@bwitham
Copy link
Contributor

bwitham commented Jun 13, 2016

sure, no problem. will do shortly.

@bwitham
Copy link
Contributor

bwitham commented Jun 13, 2016

Looks good. Looks like you removed all the wps refs.

@mikejeffe
Copy link

@bwitham should I just merge this into develop?

@bwitham
Copy link
Contributor

bwitham commented Jun 13, 2016

that's fine.

brianhatchl added a commit that referenced this issue Jan 10, 2017
  > Merge pull request #821 from ngageoint/775
  > Merge pull request #820 from ngageoint/644-2
bwitham pushed a commit that referenced this issue Jan 13, 2017
  > Merge pull request #821 from ngageoint/775
  > Merge pull request #820 from ngageoint/644-2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants