-
Notifications
You must be signed in to change notification settings - Fork 0
Update UI #37
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
Update UI #37
Conversation
beckermarc
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for doing this! I also had this on my list :)
Some comments on changes that I would like to keep / we should rather add as well to the Node.js variant
👍 yes, I also noticed that while there are quite a few updates on nodejs side, other parts are still leftovers from sflight. I'll open a corresponding PR on nodejs xtravels after merging this one |
Update UI to match the updates in https://github.com/capire/xtravels
This also includes model changes.
The node.js xtravels version contains exportJSON and exportCSV custom actions - I excluded them from this PR.