X Tutup
Skip to content

fix(angular1-router): add missing wrapper methods#6861

Closed
petebacondarwin wants to merge 1 commit intoangular:masterfrom
petebacondarwin:router-fixes
Closed

fix(angular1-router): add missing wrapper methods#6861
petebacondarwin wants to merge 1 commit intoangular:masterfrom
petebacondarwin:router-fixes

Conversation

@petebacondarwin
Copy link
Copy Markdown
Contributor

No description provided.

@petebacondarwin petebacondarwin added type: bug/fix cla: yes comp: router action: review The PR is still awaiting reviews from at least one requested reviewer labels Feb 3, 2016
@brandonroberts
Copy link
Copy Markdown
Contributor

Can you add the missing charCodeAt function to the StringWrapper also? See #6763

@petebacondarwin
Copy link
Copy Markdown
Contributor Author

Will do

@petebacondarwin
Copy link
Copy Markdown
Contributor Author

Added

@brandonroberts
Copy link
Copy Markdown
Contributor

👍

@petebacondarwin petebacondarwin changed the title fix(angular1-router): add missing ListWrapper method fix(angular1-router): add missing wrapper methods Feb 3, 2016
@btford
Copy link
Copy Markdown
Contributor

btford commented Feb 4, 2016

LGTM, merge.

@btford btford added pr_state: LGTM action: merge The PR is ready for merge by the caretaker labels Feb 4, 2016
@IgorMinar IgorMinar closed this in 55122cd Feb 9, 2016
@petebacondarwin petebacondarwin deleted the router-fixes branch September 2, 2017 14:44
@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 12, 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 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.

3 participants

X Tutup