-
Notifications
You must be signed in to change notification settings - Fork 5
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
Added the MapRemoteFloret #220
Conversation
Generated by 🚫 Danger |
0d7d30d
to
4219b6f
Compare
4219b6f
to
e614401
Compare
9d751d8
to
7aef63b
Compare
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.
LGTM. Honestly, I really like the code!
Here are my nitpicks on some wording :D I'm not 100% sure about some, though. I just thought we could simplify it here and there. What do you think?
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.
Closing backticks 🤓
3367aa9
to
a8901f9
Compare
Oh man. How embarrassing. Thanks! 🙇 |
This PR adds the MapRemoteFloret and thus implements the first step towards #216.
I did not yet implement the XML parsing yet and wanted to open up the discussion on how we should do that.