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

Editorial fixes to v1.1.1 repository #13

Open
wants to merge 94 commits into
base: master
Choose a base branch
from
Open

Editorial fixes to v1.1.1 repository #13

wants to merge 94 commits into from

Conversation

rabardini
Copy link

From Mitch:

  1. oic.rule-schema.json (Bug 1146)

"links": {
"type": "array",
"description": "Array of OIC web links that are the rule members, this is the script",
"items" : {
"allOf": [
{ "$ref": "oic.oic-link-schema.json#/definitions/oic.oic-link#" },
{ "required" : [ "ins" ] }
]
}
}
},

The “#” need to be removed.

  1. oic.types-schema.json

"uuid": {
"description": "An identifier formatted according to IETF RFC 4122.",
"type":"string",
"format": "date-time",
"pattern": "^[a-fA-F0-9]{8}-[a-fA-F0-9]{4}-[a-fA-F0-9]{4}-[a-fA-F0-9]{4}-[a-fA-F0-9]{12}$"
},

I can’t recall if we agreed to keep the “format” for “uuid” but if we did, it should not be “date-time”.

  1. oic.oic-link-schema.json

"if": {
"type": "array",
"items": [
{
"type" : "string",
"enum" : ["oic.if.baseline", "oic.if.ll", "oic.if.b", "oic.if.lb", "oic.if.rw", "oic.if.r", "oic.if.a", "oic.if.s" ]
}
],
"minItems": 1,
"readOnly": true,
"description": "The interface set supported by this resource"
},

oic.if.lb was removed from the core spec.

  1. oic.oic-link-shema.json

"sec": {
"readOnly": true,
"description": "Specifies if security needs to be turned on when looking to interact with the Resource",
"type": "boolean"
},

Per CR 48, we had agreed that the default value of “sec” is false. The schema file use to include: “default”: false. Do we no longer include the default value in the schema file?

stephanesan and others added 30 commits February 12, 2016 11:05
This is needed as the raml2doc tool has an issue with '©'.
This is needed as the raml2doc tool has an issue with '©'.
Each directory needs to contain both the License and the Disclaimer.
- Add License and Disclaimer to schema directory.
Each file needs to contain the full copyright text.
- Incorporate the copyright text instead of including the file.
Fixed the files that didn't have the 'schemas' subfolder in the id.
Fixed the files that didn't have the '#' at the end of the id that
indicates a same document reference.
Various fixes on id's and incorporate legal content through includes.
Changes to various schema files to reflect alignment to OIC v1.1
reflected in various OIC Change Requests.
Align schemas to OIC v1.1 changes
Merge pull request #2 from rabardini/refactor
Merge Refactor onto Master branch
v1.1.0 Core Specification RAML/JSON files
rabardini and others added 30 commits October 11, 2016 11:37
Fix "data" format type and fix collections. These where changes discussed and accepted during the Architecture 10/10/2016 teleconference.
As per the discussion in atg October 24, 2016, removed link tagging and
added the "rep" keyword to the batch representation
The "rel" parameter needs to also be a multi value (repeatable) array
as per RFC6690
made "di" and "ins" use reference to UUID definition instead of pattern
property
fix typo in JSON that the editor didn't flag
readonly only applies to resource properties, not link parameter
definitions.

The "links" property of a collection resource is array of links with
parameters. The links property is already made readonly through the
oic.if.ll interface.
Removed the link tag and added "rep" keyword
Fix for missing closing bracket in oic.collection.batch-retrive-schema.json
Fix for invalid json schema
Fix allOf section in oic.collection-schema.json
Allow the "rel" link attribute to be either string or array
all resource definitions need additionalproperties to be true for
validation of composite resources
Fix for invalid json of oic.oic-link-schema.json
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants