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

issues with extensions to <postal> #137

Closed
3 tasks
reschke opened this issue May 3, 2020 · 8 comments
Closed
3 tasks

issues with extensions to <postal> #137

reschke opened this issue May 3, 2020 · 8 comments

Comments

@reschke
Copy link
Contributor

reschke commented May 3, 2020

The motivation for this incredible complex feature (see related tickets on xml2rfc trac), as explained in https://tools.ietf.org/html/draft-levkowetz-xml2rfc-v3-implementation-notes-10#section-3.1.13, is IMHO very weak.

It would be far easier to remove the inconsistency with the HTML format spec by fixing that spec instead. The use case of making statistics about the countries authors come from (if really needed) could be addressed by a for more simpler extension.

If the extensions are to stay we need to:

@jrlevine
Copy link
Contributor

jrlevine commented May 7, 2020

I agree, if the problem is that the order of address lines is different in English and other languages, there are much simpler ways to solve it.

@reschke
Copy link
Contributor Author

reschke commented May 8, 2020

We actually solved that by introducing https://greenbytes.de/tech/webdav/rfc7991.html#element.postalLine

@reschke
Copy link
Contributor Author

reschke commented May 13, 2020

mnot referenced this issue in mnot/v3grammar Dec 29, 2020
@stpeter
Copy link
Collaborator

stpeter commented Jul 14, 2021

Based on rfc-interest discussion, if we move to postalLine plus optional country then that would simplify processing. We'd still need to document that in 7991bis, of course.

@reschke
Copy link
Contributor Author

reschke commented Jul 14, 2021

I can prepare a PR if you want me to.

@jrlevine
Copy link
Contributor

@reschke please do
This requires a grammar change, the first one in quite a while. Maybe we should wait and see if there are others we can batch with it.

@reschke
Copy link
Contributor Author

reschke commented Sep 1, 2021

Here you go: #204

@jrlevine
Copy link
Contributor

merged manually

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

No branches or pull requests

3 participants