Update shiny_attributes() to support attribute-level annotations about eml HOT 5 CLOSED

Sounds good -- PRs are always welcome!

I do wonder if it wouldn't make more sense to package the shiny app components as a separate package though? I'd be happy to re-import the methods here if that's helpful. The shiny app code so far has all been community-contributed!

from eml.

It might indeed make sense to separate it, but its pretty convenient where it is. Some folks at NCEAS that were doing a lot of data curation with this module contributed it here for convenience. But as a UI function its pretty different from the other functions. I could also see this being a really helpful RStudio extension package.

Adding the annotation fields looks like a great idea. We have some other work on providing annotations in EML via R and via web interfaces, and so it would be great to get perspectives from @amoeba, @jeanetteclark, @laijasmine, and @samanthacsik about what would be helpful here.

from eml.

I think at the moment it would be great to integrate this with the package right now as the Arctic Data Center team will really benefit from this. The current infrastructure makes it difficult to edit attributes without losing all the annotations as @BrennieDev described above with the get_attributes() -> shiny_attributes() -> set_attributes() workflow. I don't know if there are many cases where we would ever use shiny_attributes without the rest of the EML functions for us.

from eml.

In the meantime, I've created a PR anyone can take a look at to see the changes (should be pretty straightforward but let me know of any questions/concerns):
#319

from eml.

Closed with merge of PR #319

from eml.