Fixed app not working with the newest api change which has multiple reminders #19
No reviewers
Labels
No Label
bug
duplicate
enhancement
help wanted
invalid
Kind: Planning
Priority: Critical
Priority: High
Priority: Low
Priority: Medium
question
Status: Abandoned
Status: Accepted
Status: Available
Status: Blocked
Status: Completed
Status: In Progress
Status: On Hold
Status: Pending
Status: Ready for Review
Status: Review Needed
Status: Revision Needed
Type: Bug
Type: Build
Type: Enhancement
Type: Maintenance
Type: Question
wontfix
No Milestone
No Assignees
2 Participants
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: vikunja/app#19
Loading…
Reference in New Issue
No description provided.
Delete Branch "fix/latest-api-reminder-dates"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
@ -35,3 +36,3 @@
'updated': updated?.millisecondsSinceEpoch,
'created': created?.millisecondsSinceEpoch,
'reminderDate': reminder?.millisecondsSinceEpoch,
'reminderDates': reminders?.toList(),
The reminders should be transfered into an unix timestamp.
Use something like that:
Fixed.
@ -82,1 +83,4 @@
}
List<DateTime> makeDateTimesFromJSON(List<dynamic> json) {
simply use the map function:
@ -14,6 +14,7 @@ dependencies:
# Use with the CupertinoIcons class for iOS style icons.
cupertino_icons: ^0.1.2
flutter_secure_storage: 3.1.1
http: 0.12.0
Why do you add this dependency?
Because the build would otherwise fail. Apparantly this was changed in the latest flutter version.
Fair enough
CI failed, please fix! Otherwise LGTM
CI failed, please fix.
CI fixed