-
Notifications
You must be signed in to change notification settings - Fork 8
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] Refactor CreateBuffersAroundPointsView.Model
#213
Conversation
Shared/Samples/Create buffers around points/CreateBuffersAroundPointsView.swift
Show resolved
Hide resolved
Shared/Samples/Create buffers around points/CreateBuffersAroundPointsView.swift
Outdated
Show resolved
Hide resolved
Shared/Samples/Create buffers around points/CreateBuffersAroundPointsView.swift
Outdated
Show resolved
Hide resolved
Shared/Samples/Create buffers around points/CreateBuffersAroundPointsView.swift
Outdated
Show resolved
Hide resolved
Shared/Samples/Create buffers around points/CreateBuffersAroundPointsView.swift
Outdated
Show resolved
Hide resolved
Shared/Samples/Create buffers around points/CreateBuffersAroundPointsView.swift
Outdated
Show resolved
Hide resolved
Shared/Samples/Create buffers around points/CreateBuffersAroundPointsView.swift
Show resolved
Hide resolved
Shared/Samples/Create buffers around points/CreateBuffersAroundPointsView.swift
Outdated
Show resolved
Hide resolved
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.
Looks good. Just one suggestion.
Shared/Samples/Create buffers around points/CreateBuffersAroundPointsView.swift
Outdated
Show resolved
Hide resolved
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.
Meant to approve with one suggestion.
…dPointsView.swift Co-authored-by: Phil Gruenler <pgruenler@esri.com>
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.
Looks good.
Description
Close #212. This PR updates
Create buffers around points
added in #185 .Linked Issue(s)
How To Test
The sample should behave the same as before.
To Discuss
In hindsight I should have paid more attention to the code review. This sample was one of Caleb's first samples so I let it go without being too harsh.
Ideally we should have a more careful mindset on how to use the data model object. Try to avoid the case where the model knows about states in the UI, or in other words, UI states should be derived from the model, not reverse.