-
Notifications
You must be signed in to change notification settings - Fork 870
docs(tutorial): add dart version of toh tutorial #700
Conversation
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.
should be Tires instead of constructor
|
Oh oopz @zoechi I have only converted toh-1 up to toh-4 to dart. I first just copied all the ts source files from the ts directory to the dart directory ,but I think the files about DI (dependency injection) shouldn't actually be here, so I removed them now (should probably also removed from the ts source). Also toh-5 is not converted, because the ts version is not yet finished. |
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.
=== not valid Dart
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.
thanks, fixed, (will push later)
|
The |
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.
TS ===
245e733 to
ebb411d
Compare
|
@zoechi I removed that file as well now, and fixed the other comments. |
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.
shouldn't this use async / await ?
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.
This is ok, it uses hero_service_1.dart where the value is returned sync.
|
I did just a shallow review but now LGTM |
|
Taking a very belated look now... |
|
@kasperpeulen could you please You can rename your .jade files to *-NEW.jade to get around some of the conflicts. And just drop the _data.json changes. |
|
Just FYI, I'm copying down the toh-1 code to submit in a separate PR. |
|
Closing, now that all changes have gotten into merged PRs. Thank you, @kasperpeulen! |
@kwalrath in this PR, I just focused on getting the Dart version up to date with the typescript version
In the other PR I ported the end result to dart, but it seems like this code was only used for plnkr, or maybe it will be used for the last part (toh-pt5), but this is not yet done for typescript it seems.