Skip to content

fix(http): change http interfaces to types - #4024

Closed
jeffbcross wants to merge 1 commit into
angular:masterfrom
jeffbcross:http-types
Closed

jeffbcross wants to merge 1 commit into
angular:masterfrom
jeffbcross:http-types

Conversation

@jeffbcross

Copy link
Copy Markdown
Contributor

Types were previously not handled correctly by the type definition
generation process, so a couple of things in http had to be changed
to interfaces. This issue has been fixed, so now they're types again.

@jeffbcross jeffbcross added type: bug/fix comp: data-access action: review The PR is still awaiting reviews from at least one requested reviewer labels Sep 7, 2015
@jeffbcross jeffbcross added this to the alpha-37 milestone Sep 7, 2015
@IgorMinar IgorMinar added pr_state: LGTM action: merge The PR is ready for merge by the caretaker labels Sep 16, 2015
@mary-poppins

Copy link
Copy Markdown

Merging PR #4024 on behalf of @IgorMinar to branch presubmit-IgorMinar-pr-4024.

1 similar comment
@mary-poppins

Copy link
Copy Markdown

Merging PR #4024 on behalf of @IgorMinar to branch presubmit-IgorMinar-pr-4024.

@mary-poppins mary-poppins removed action: merge The PR is ready for merge by the caretaker labels Sep 16, 2015
@jeffbcross jeffbcross added the action: merge The PR is ready for merge by the caretaker label Sep 18, 2015
@mary-poppins

Copy link
Copy Markdown

Merging PR #4024 on behalf of @jeffbcross to branch presubmit-jeffbcross-pr-4024.

@mary-poppins mary-poppins removed the action: merge The PR is ready for merge by the caretaker label Sep 18, 2015
Types were previously not handled correctly by the type definition
generation process, so a couple of things in http had to be changed 
to interfaces. This issue has been fixed, so now they're types again.
robwormald pushed a commit to robwormald/angular that referenced this pull request Sep 25, 2015
Types were previously not handled correctly by the type definition
generation process, so a couple of things in http had to be changed 
to interfaces. This issue has been fixed, so now they're types again.

Closes angular#4024
@angular-automatic-lock-bot

Copy link
Copy Markdown

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot Bot locked and limited conversation to collaborators Sep 7, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: review The PR is still awaiting reviews from at least one requested reviewer cla: yes type: bug/fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants