Skip to content

IANA profile should be frame not framed #132

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

Closed
VladimirAlexiev opened this issue Aug 17, 2022 · 5 comments · Fixed by #134
Closed

IANA profile should be frame not framed #132

VladimirAlexiev opened this issue Aug 17, 2022 · 5 comments · Fixed by #134
Assignees

Comments

@VladimirAlexiev
Copy link

VladimirAlexiev commented Aug 17, 2022

https://www.w3.org/TR/json-ld11-framing/#iana-considerations:

http://www.w3.org/ns/json-ld#framed
To specify a JSON-LD Frame.
The http://www.w3.org/ns/json-ld#framed SHOULD be used when serving and requesting a JSON-LD frame document.

https://w3c.github.io/json-ld-syntax/#application-ld-json

This specification defines six values for the profile parameter.
http://www.w3.org/ns/json-ld#frame
To request or specify a JSON-LD frame document.
http://www.w3.org/ns/json-ld#framed
To request or specify framed JSON-LD document form.

  • I think that in json-ld11-framing, the correct profile to use is frame not framed
@gkellogg
Copy link
Member

Thanks for noticing this. I don't recall right now why we had a separate IANA considerations section in the Framing document, as it is redundant with that from the Syntax document, and as you point out, inconsistent.

I think the corrective action is the just make the Framing document consistent with the Syntax document for this particular profile parameter in an Erratum.

@gkellogg gkellogg self-assigned this Aug 18, 2022
@gkellogg gkellogg added the ErratumRaised Used by the errata management label Aug 18, 2022
gkellogg added a commit that referenced this issue Aug 19, 2022
@gkellogg gkellogg mentioned this issue Aug 19, 2022
@gkellogg
Copy link
Member

Summary: Update IANA considerations to make thehttp://www.w3.org/ns/json-ld#frame parameter required and change from http://www.w3.org/ns/json-ld#frame, which was inaccurate and inconsistent with the same definition in the with the narrative text and with the same definition in JSON-LD11.

gkellogg added a commit that referenced this issue Aug 22, 2022
@gkellogg gkellogg added Errata Used by the errata management and removed ErratumRaised Used by the errata management labels Aug 22, 2022
@gkellogg
Copy link
Member

I'm re-opening this to undo the normative change to the ReSpec source to allow for future publication without going through the CR cycle again.

@gkellogg gkellogg reopened this Oct 15, 2022
@gkellogg gkellogg added the defer-future-version Defer this issue until a future version of JSON-LD label Oct 15, 2022
@gkellogg
Copy link
Member

The lines noted in #137 should be added back when ready for a new CR phase.

@gkellogg gkellogg removed the defer-future-version Defer this issue until a future version of JSON-LD label Jul 15, 2023
gkellogg added a commit that referenced this issue Aug 8, 2023
…ther than optinoal, and use "frame" instead of "framed".

For #132.
gkellogg added a commit that referenced this issue Oct 30, 2023
…ther than optinoal, and use "frame" instead of "framed".

For #132.
@VladimirAlexiev
Copy link
Author

hi @gkellogg I'm closing this based on your latest commit. Please reopen if I'm wrong

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants