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

JSON-LD compatible profile #70

Closed
wants to merge 4 commits into from

Conversation

rob-metalinkage
Copy link
Contributor

@rob-metalinkage rob-metalinkage commented Aug 3, 2022

Initial statements about JSON-LD compatible profile


Preview | Diff

Initial statements about JSON-LD compatible profile
@gkellogg gkellogg changed the title Update index.html JSON-LD compatible profile Aug 3, 2022
@gkellogg
Copy link
Member

gkellogg commented Aug 3, 2022

Thanks, we’ll try to discuss in the call tomorrow.

spec/index.html Outdated Show resolved Hide resolved
spec/index.html Outdated Show resolved Hide resolved
spec/index.html Outdated Show resolved Hide resolved
@gkellogg
Copy link
Member

gkellogg commented Aug 3, 2022

This begs the question of what a profile that wasn't compatible with JSON-LD would look like, and how it could be processed. The only real features I could see as being useful are a multiple-document model (for which there are JSON analogs) and support for non-JSON tags (such as datetime, ...). I'm not sure we can make strong statements on a YAML-LD-JSON (or as I've called it "basic") profile without fleshing out what the goals of a more extended profile might be.

@gkellogg
Copy link
Member

gkellogg commented Aug 3, 2022

This PR was discussed on the Aug 03 meeting where we decided to hold off until we had a better understanding of what an extended profile might do beyond a basic JSON-compatible profile.

Co-authored-by: Ted Thibodeau Jr <tthibodeau@openlinksw.com>
@gkellogg
Copy link
Member

This issue was discussed in the August 17th meeting.

General feel that it can be merged, but more detail is still necessary.

gkellogg and others added 2 commits August 17, 2022 13:39
Co-authored-by: Ted Thibodeau Jr <tthibodeau@openlinksw.com>
gkellogg added a commit that referenced this pull request Aug 17, 2022
@gkellogg gkellogg mentioned this pull request Aug 17, 2022
@gkellogg
Copy link
Member

Replaced via #74.

@gkellogg gkellogg closed this Aug 17, 2022
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.

4 participants