Fixed app not working with the newest api change which has multiple reminders #19

konrad · 2018-12-02T16:03:39Z

konrad commented

2018-12-02 16:03:39 +00:00

No description provided.

JonasFranz was assigned by konrad

2018-12-02 16:03:44 +00:00

JonasFranz requested changes 2018-12-03 09:54:17 +00:00

lib/models/task.dart Outdated

						
				@ -35,3 +36,3 @@

				        'updated': updated?.millisecondsSinceEpoch,

				        'created': created?.millisecondsSinceEpoch,

				        'reminderDate': reminder?.millisecondsSinceEpoch,

				        'reminderDates': reminders?.toList(),

JonasFranz commented

2018-12-03 09:52:43 +00:00

The reminders should be transfered into an unix timestamp.

Use something like that:

reminders?.map((date) => date.millisecondsSinceEpoch)?.toList()

The reminders should be transfered into an unix timestamp. Use something like that: ```dart reminders?.map((date) => date.millisecondsSinceEpoch)?.toList() ```

konrad commented

2018-12-03 12:52:22 +00:00

Fixed.

lib/models/task.dart Outdated

						
				@ -82,1 +83,4 @@

				}

				List<DateTime> makeDateTimesFromJSON(List<dynamic> json) {

JonasFranz commented

2018-12-03 09:53:56 +00:00

simply use the map function:

json?.map((milli) => DateTime.fromMillisecondsSinceEpoch(milli))?.toList()

simply use the map function: ```dart json?.map((milli) => DateTime.fromMillisecondsSinceEpoch(milli))?.toList() ```

pubspec.yaml

						
				@ -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