-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Remove unneeded update tracking. #113
Conversation
@@ -5,10 +5,10 @@ import Prelude | |||
import ReactiveSwift | |||
import Result | |||
|
|||
public struct UpdateData { | |||
private struct UpdateData { |
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.
oh dang actually you don't even need to use this struct anymore since this context
was for the unneeded events sry
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.
lol
public let project: Project | ||
public let update: Update | ||
public let context: Koala.UpdateContext | ||
fileprivate struct UpdateData { |
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.
you could also kill this UpdateData
struct entirely and just use a pair for project and update if your heart desires :)
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.
yah for sure but just wanna do the minimal work right now. or you can clean up your own mess ;)
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.
༼ຈل͜ຈ༽
The pr #103 added some tracking to updates that isn't needed since it's done on the server side. So just removing that and correcting a few small things.