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

add TYPE property to EXT element #90

Closed
mamund opened this issue Sep 19, 2020 · 4 comments
Closed

add TYPE property to EXT element #90

mamund opened this issue Sep 19, 2020 · 4 comments

Comments

@mamund
Copy link
Member

mamund commented Sep 19, 2020

i've found a handful of times that i need to mark EXT elements with TYPE information to help reader/writer tooling. This will be a single string property that can contain any valid string text. the meaning of that value can be documented when documenting the EXT element itself.

<ext id="hatSize" type="enum" value="large medium small really-big" />

this may also help with issue #8

@mamund mamund added this to the draft-04 milestone Sep 19, 2020
@mamund mamund self-assigned this Sep 19, 2020
@filip26
Copy link

filip26 commented Oct 23, 2020

If type value range is not defined by the specification then it adds just a little value, because it is expected that a client/processor 'knows' but if a client knows then it has to know an unspecified type attribute as well.

if the type value range is defined then I'm thinking maybe introducing other elements could be more readable.

e.g.

<enum id="hatSize" value="...."/>

I personally like keeping the spec simple. Let's discuss it.

@mamund
Copy link
Member Author

mamund commented Nov 4, 2020

we can discuss this in the weekly AOOH. I'm OK w/ dropping the TYPE element for EXT if it complicates things.

@mamund
Copy link
Member Author

mamund commented Nov 5, 2020

decided to not add this right now. not a wide need and we can address late, if needed.

@mamund mamund closed this as completed Nov 10, 2020
@mamund
Copy link
Member Author

mamund commented Nov 10, 2020

closed out

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

No branches or pull requests

2 participants