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

Include Appendix A media type registration #168

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

guybedford
Copy link
Collaborator

@guybedford guybedford commented Jul 25, 2019

This includes the media type registration as an Appendix A in the specification text, according to the process for W3C registrations outlined at https://www.w3.org/2002/06/registering-mediatype2014.html.

I have included myself as the contact for further information for now, but would be happy to change this if another contact would like to be listed - I just didn't want to incorrectly assume this responsibility for anyone else.


Preview | Diff

@guybedford guybedford requested a review from domenic July 25, 2019 21:14
:: N/A
: Object Identifiers:
:: N/A
: Person & email address to contact for further information:

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@wseltzer / @plehegar - Thoughts on the right contact person here?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't recommend putting individual email address there since it needs to survive individuals moving on but I have no objection if you folks feels otherwise. At the end of the day, the best way to reach out is through GitHub as far as I know.
Basically, we'd need an email address that someone can see and redirect to the right person if needed. Some ideas, with various pros/cons:

Copy link
Collaborator

@domenic domenic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You may want to consult https://html.spec.whatwg.org/#application/microdata+json which illustrates how you can (should?) defer a lot to application/json.

spec.bs Outdated Show resolved Hide resolved
spec.bs Outdated Show resolved Hide resolved
spec.bs Outdated Show resolved Hide resolved
spec.bs Outdated Show resolved Hide resolved
spec.bs Outdated Show resolved Hide resolved
spec.bs Show resolved Hide resolved
spec.bs Outdated Show resolved Hide resolved
spec.bs Outdated Show resolved Hide resolved
spec.bs Outdated Show resolved Hide resolved
spec.bs Outdated Show resolved Hide resolved
@guybedford
Copy link
Collaborator Author

I've included all the feedback here now. Note that for both the required parameters and optional parameters, referring to the JSON specification is effectively referring to the N/A values. Using an N/A field is very specifically requested to be provided for non-applicable fields in the template, so directly providing this value seems better. I did refer to the JSON spec for the Mac filetype code though, being the only one of these references that is not N/A.

Let me know if there is any further feedback to merge further.

@guybedford
Copy link
Collaborator Author

The CI error here seems to be an unrelated issue with the lockfile in the reference implementation.

@yoavweiss yoavweiss changed the base branch from master to main March 5, 2021 21:39
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