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

Added the MapRemoteFloret #220

Merged
merged 6 commits into from
Mar 15, 2021
Merged

Added the MapRemoteFloret #220

merged 6 commits into from
Mar 15, 2021

Conversation

brototyp
Copy link
Member

@brototyp brototyp commented Mar 13, 2021

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.

Simulator Screen Shot - iPod touch (7th generation) - 2021-03-13 at 15 20 31
Simulator Screen Shot - iPod touch (7th generation) - 2021-03-13 at 15 20 34

@github-actions
Copy link

github-actions bot commented Mar 13, 2021

2 Warnings
⚠️ Are there any changes that should be explained in the README.md?
⚠️ Cauli/CauliURLProtocol/CauliURLProtocol.swift#L79 - Arguments can be omitted when matching enums with associated values if they are not used.
empty_enum_arguments CauliURLProtocol.swift:79

Generated by 🚫 Danger

@brototyp brototyp force-pushed the feature/216-map-remote-floret branch from 0d7d30d to 4219b6f Compare March 13, 2021 14:29
@github-actions github-actions bot deleted a comment from brototyp-bot Mar 13, 2021
@github-actions github-actions bot deleted a comment from brototyp-bot Mar 13, 2021
@github-actions github-actions bot deleted a comment from brototyp-bot Mar 13, 2021
@github-actions github-actions bot deleted a comment from brototyp-bot Mar 13, 2021
@github-actions github-actions bot deleted a comment from brototyp-bot Mar 13, 2021
@github-actions github-actions bot deleted a comment from brototyp-bot Mar 13, 2021
@brototyp brototyp force-pushed the feature/216-map-remote-floret branch from 4219b6f to e614401 Compare March 13, 2021 14:34
@brototyp brototyp force-pushed the feature/216-map-remote-floret branch from 9d751d8 to 7aef63b Compare March 13, 2021 14:40
@github-actions github-actions bot deleted a comment from brototyp-bot Mar 13, 2021
@github-actions github-actions bot deleted a comment from brototyp-bot Mar 13, 2021
@github-actions github-actions bot deleted a comment from brototyp-bot Mar 13, 2021
@github-actions github-actions bot deleted a comment from brototyp-bot Mar 13, 2021
@github-actions github-actions bot deleted a comment from brototyp-bot Mar 13, 2021
Copy link
Collaborator

@lukaswuerzburger lukaswuerzburger left a 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?

Cauli/Florets/MapRemote/MappingsListDatasource.swift Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
Cauli/Florets/MapRemote/MapRemoteFloret.swift Outdated Show resolved Hide resolved
Cauli/Florets/MapRemote/MapRemoteFloret.swift Outdated Show resolved Hide resolved
Cauli/Florets/MapRemote/MapRemoteFloret.swift Outdated Show resolved Hide resolved
Cauli/Florets/MapRemote/Mapping.swift Outdated Show resolved Hide resolved
Cauli/Florets/MapRemote/Mapping.swift Outdated Show resolved Hide resolved
Cauli/Florets/MapRemote/Mapping.swift Outdated Show resolved Hide resolved
Cauli/Florets/MapRemote/Mapping.swift Outdated Show resolved Hide resolved
docs/guides/Florets.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@lukaswuerzburger lukaswuerzburger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Closing backticks 🤓

Cauli/Florets/MapRemote/MapRemoteFloret.swift Outdated Show resolved Hide resolved
Cauli/Florets/MapRemote/MapRemoteFloret.swift Outdated Show resolved Hide resolved
@brototyp brototyp force-pushed the feature/216-map-remote-floret branch from 3367aa9 to a8901f9 Compare March 14, 2021 14:05
@brototyp
Copy link
Member Author

Closing backticks 🤓

Oh man. How embarrassing. Thanks! 🙇

@brototyp brototyp merged commit 62626f4 into develop Mar 15, 2021
@brototyp brototyp deleted the feature/216-map-remote-floret branch March 15, 2021 08:36
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.

3 participants