ioos / compliance-checker-web

Web-based front-end for the IOOS Compliance Checker project
https://compliance.ioos.us
MIT License
5 stars 10 forks source link

Add description and references #37

Closed Bobfrat closed 8 years ago

Bobfrat commented 8 years ago

@lukecampbell please review. This addresses https://github.com/ioos/compliance-checker/issues/291

screen shot 2016-09-07 at 2 09 24 pm
lukecampbell commented 8 years ago

This is a good start but we should go through the individual tests and plugins and make sure that the descriptions are appropriate. I know that the original issue @rsignell-usgs raised isn't addressed by this because the description for the NCEI checks is still completely vague.

screen shot 2016-09-07 at 2 34 49 pm

Bobfrat commented 8 years ago

@lukecampbell agreed, the reference link to NCEI NetCDF templates help but the _cc_description attributes should be updated. In fact the Glider DAC plugin doesn't have a description at all.

MathewBiddle commented 8 years ago

NCEI can help come up with better language for the NCEI checks.

Bobfrat commented 8 years ago

@mbiddle-nodc that would be awesome.

A search for _cc_description in the https://github.com/ioos/cc-plugin-ncei repo shows where all the descriptions are currently defined. You can pass along the descriptions to me and I'll make the updates or just make a PR to that repo directly.

https://github.com/ioos/cc-plugin-ncei/search?utf8=%E2%9C%93&q=_cc_description

MathewBiddle commented 8 years ago

This might take a little bit to get the language together. @Bobfrat I'll notify you once we have all the descriptions ready.

MathewBiddle commented 8 years ago

I'm not very fluent in github, but I think I made the changes and put the pull requests in. Let me know if it didn't work.

Bobfrat commented 8 years ago

👍 Looks good I'll have @lukecampbell review and merge. Thanks!