Skip to content
This repository has been archived by the owner on Feb 22, 2018. It is now read-only.

Recording backend url #872

Closed

Conversation

kurman-zz
Copy link
Contributor

Moving '/record' url to PlaybackHttpBackendConfig, should make overriding the value from app code easier.

@kurman-zz kurman-zz added cla: yes and removed cla: no labels Apr 8, 2014
@@ -12,6 +12,9 @@ import 'package:angular/playback/playback_data.dart' as playback_data;

@NgInjectableService()
class PlaybackHttpBackendConfig {

String get recorder_url => '/record';
Copy link
Contributor

Choose a reason for hiding this comment

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

recoderUrl

@vicb
Copy link
Contributor

vicb commented Apr 8, 2014

@kurman please squash the commits and follow the guidelines in CONTRIBUTING.md for the commit comment. Thanks.

mhevery pushed a commit that referenced this pull request Apr 8, 2014
Moving hardcoded recorder URL to config class and can be overridden by installing extended PlaybackHttpBackendConfig class in applications.

Closes #872
mhevery pushed a commit to mhevery/angular.dart that referenced this pull request Apr 9, 2014
Moving hardcoded recorder URL to config class and can be overridden by installing extended PlaybackHttpBackendConfig class in applications.

Closes dart-archive#872
@mhevery mhevery closed this in 6ecf1d5 Apr 10, 2014
@kurman-zz
Copy link
Contributor Author

Thank you Mishko for taking care of it, and Victor for the feedback.

Kurman
On Apr 9, 2014 10:03 PM, "Miško Hevery" [email protected] wrote:

Closed #872 #872 via 6ecf1d56ecf1d5
.


Reply to this email directly or view it on GitHubhttps://github.com//pull/872
.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Development

Successfully merging this pull request may close these issues.

3 participants