Skip to content
This repository was archived by the owner on Dec 4, 2017. It is now read-only.

docs(server-communication): Resolve todo #1993

Merged
merged 2 commits into from
Aug 1, 2016
Merged

docs(server-communication): Resolve todo #1993

merged 2 commits into from
Aug 1, 2016

Conversation

adaojunior
Copy link
Contributor

No description provided.

@chalin
Copy link
Contributor

chalin commented Jul 28, 2016

Thanks, LGTM

@@ -11,7 +11,7 @@ dependencies:
http: ^0.11.3+3
jsonpadding: ^0.1.0
stream_transformers: ^0.3.0+3
http_in_memory_web_api: ^0.0.1
http_in_memory_web_api: ^0.2.0
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is this, @chalin ? Otherwise this PR makes sense to me.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@wardbell : there was a bug in http_in_memory_web_api which the author fixed. This PR removes the temporary workaround and bumps up the library version to get the fixed version.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what is http_in_memory_web_api? I guess this is all Dart so I shouldn't care ;-)

Copy link
Contributor Author

@adaojunior adaojunior Aug 1, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@wardbell this is a port of a2-in-memory-web-api that you wrote.

http-in-memory-web-api

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, it is an all-Dart counterpart.

@kwalrath
Copy link
Contributor

kwalrath commented Aug 1, 2016

I'll merge this now unless someone screams...

@kwalrath kwalrath merged commit 6f63e6a into angular:master Aug 1, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants