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

"field_order"=<integer> for message fields and frame layers #3

Open
dodohand opened this issue Apr 27, 2022 · 2 comments
Open

"field_order"=<integer> for message fields and frame layers #3

dodohand opened this issue Apr 27, 2022 · 2 comments

Comments

@dodohand
Copy link

Hello,

The Setup:

I ran right into the issue you've raised as pantor/inja#213... but wouldn't like to stop and wait for an inja fix.

The Punchline:

Have you considered a key:value pair like "field_order"=0 as part of the message fields and the frame layers so that
they can be put into the intended/required order in documentation? This would also go toward addressing the basic
fact that JSON is unordered by definition. I'm not sure that inja would permit a nice syntax for taking advantage of
this field, but it should allow successful parsing of the JSON in more-expressive languages.

@mathisloge
Copy link
Owner

Hey, thanks for highlighting. I'll look into this tomorrow in more detail 👍

mathisloge added a commit that referenced this issue Apr 28, 2022
@mathisloge
Copy link
Owner

@dodohand I've added protodocOrder to each field. It will have the count index as the member vector in commsdsl (as specified in the schema xml.)

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

2 participants