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 tips-and-tricks page with notes on XML parsing #7

Merged
merged 1 commit into from
Feb 4, 2023
Merged

Conversation

johanmalm
Copy link
Contributor

...further to this discussion

labwc/labwc#735 (comment)

@Consolatis - does it read okay?

@Consolatis
Copy link
Member

Yes looks about right

@johanmalm johanmalm merged commit abe9e34 into main Feb 4, 2023
@Consolatis
Copy link
Member

I've just read that page again:

Do we want to remove the To the end-user, part?

@johanmalm johanmalm deleted the xml branch February 19, 2023 14:41
@johanmalm
Copy link
Contributor Author

Just those three words? Sure. I’m out of action for the next few days, but feel free 😄

Or were you thinking the examples too?

@Consolatis
Copy link
Member

Or were you thinking the examples too?

I think the examples are perfectly fine, in my mind the end-user part just reads a bit strange.

Just those three words? Sure. I’m out of action for the next few days, but feel free smile

Can't accept PR's in this repo, didn't try to just push though.

@johanmalm
Copy link
Contributor Author

I can merge from phone 😄 if needed

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.

2 participants