Skip to content

fix(Headers): serializable toJSON#6714

Closed
PatrickJS wants to merge 1 commit into
angular:masterfrom
PatrickJS:headers-to-json
Closed

fix(Headers): serializable toJSON#6714
PatrickJS wants to merge 1 commit into
angular:masterfrom
PatrickJS:headers-to-json

Conversation

@PatrickJS
Copy link
Copy Markdown
Contributor

fixes #6073

@hansl
Copy link
Copy Markdown
Contributor

hansl commented Feb 1, 2016

Could you rebase this CL over master please?

@hansl
Copy link
Copy Markdown
Contributor

hansl commented Feb 1, 2016

LGTM, with the caveat that the dart_ddc test passes.

@PatrickJS
Copy link
Copy Markdown
Contributor Author

@hansl I rebased, I'm not sure why dart_ddc fails

@hansl hansl added pr_state: LGTM action: merge The PR is ready for merge by the caretaker labels Feb 1, 2016
@hansl
Copy link
Copy Markdown
Contributor

hansl commented Feb 1, 2016

Okay Dart DDC just passed. I'm going to stamp this. Good work!

@PatrickJS
Copy link
Copy Markdown
Contributor Author

kool, thanks

@mary-poppins
Copy link
Copy Markdown

Merging PR #6714 on behalf of @alexeagle to branch presubmit-alexeagle-pr-6714.

@mhevery mhevery closed this in b55f176 Feb 3, 2016
@PatrickJS PatrickJS deleted the headers-to-json branch February 3, 2016 15:11
@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: merge The PR is ready for merge by the caretaker cla: yes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

HTTP Headers .toJSON() only returns values (not keys) of the headers

5 participants