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.

Sign up for GitHub

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

Jump to bottom

Update to ACDD 1.3 #25

Open
lesserwhirls opened this issue Sep 13, 2016 · 4 comments
Open

Update to ACDD 1.3 #25

lesserwhirls opened this issue Sep 13, 2016 · 4 comments
Labels
enhancement

Comments

@lesserwhirls
Copy link
Collaborator

Try simply updating threddsIso/src/main/resources/xsl/nciso/UnidataDDCount-HTML.xsl with transforms/UnidataDDCount-HTML.xsl from @noaaroland
@lesserwhirls
Copy link
Collaborator Author

Implementation questions:

  • Should this target TDS v4.6 or v5.0? I would suggest upgrade for 5.0, make 1.3 the default, and allow user to decide which version to use.
  • Should we provide a way for users to provide their own xslt?

@geoneubie
Copy link
Contributor

1.3 default makes sense to me for TDS 5.0. Users can already provider their own xslt by overwriting the default xslt(s) located in the resources folder of their TDS installation, but perhaps you're thinking of an additive solution?

@lesserwhirls
Copy link
Collaborator Author

Users can already provider their own xslt by overwriting the default xslt(s) located in the
resources folder of their TDS installation

@geoneubie - do you mean overwriting the file in the exploded war directory?

@geoneubie
Copy link
Contributor

@lesserwhirls yes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@lesserwhirls @geoneubie