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

onlineResource 2.0 #50

Closed
stansmith907 opened this issue Aug 23, 2016 · 6 comments
Closed

onlineResource 2.0 #50

stansmith907 opened this issue Aug 23, 2016 · 6 comments
Labels
enhancement Improve or modify an existing feature
Milestone

Comments

@stansmith907
Copy link
Contributor

stansmith907 commented Aug 23, 2016

Changes:

  1. “protocolRequest” was added to onlineResource but not used in mdJson 2.0. Definition: “request used to access the resource depending on the protocol (to be used mainly for POST requests).” No changes from mdJson 1.0.
            "onlineResource": [
                {
                    "uri": "http://thisisanexample.com",
                    "protocol": "",
                    "name": "",
                    "description": "",
                    "function": ""
                }
            ],

see XML example responsibleParty -3.xml

@stansmith907 stansmith907 added the enhancement Improve or modify an existing feature label Aug 23, 2016
@jlblcc jlblcc modified the milestone: 2.0 Aug 23, 2016
This was referenced Aug 31, 2016
jlblcc added a commit that referenced this issue Dec 31, 2016
@stansmith907
Copy link
Contributor Author

In our effort to conform to naming standards for mdJSON elements we adopted a convention to capitalize acronyms. What if the element name is an acronym, as uri above?

  • uri
  • URI
  • uRI
  • resourceURI

@jlblcc
Copy link
Member

jlblcc commented Jan 17, 2017

Why did we change this to resourceURI? I prefer just plain uri. If the property name is a single acronym, I'd just lowercase the whole thing. I don't really care as long as it's consistent. In fact, why don't we just treat acronyms as lowercase words. That follows KISS logic.

So:

  • uri
  • resourceUri
  • resourceJsonUri
  • resourceJsonUriUsa

@stansmith907
Copy link
Contributor Author

stansmith907 commented Jan 17, 2017 via email

@jlblcc
Copy link
Member

jlblcc commented Jan 17, 2017

My bad. Do you want to leave them upper-cased?

@stansmith907
Copy link
Contributor Author

stansmith907 commented Jan 17, 2017 via email

@jlblcc
Copy link
Member

jlblcc commented Jan 17, 2017

That's why I was suggesting that we just treat them as "normal" words. Just camel-case like any other word.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Improve or modify an existing feature
Projects
None yet
Development

No branches or pull requests

2 participants