X Tutup
Skip to content

fix(router): RouterOutlet loads component twice in a race condition#7545

Closed
mhevery wants to merge 1 commit intoangular:masterfrom
mhevery:issue7497
Closed

fix(router): RouterOutlet loads component twice in a race condition#7545
mhevery wants to merge 1 commit intoangular:masterfrom
mhevery:issue7497

Conversation

@mhevery
Copy link
Copy Markdown
Contributor

@mhevery mhevery commented Mar 11, 2016

bug fix #7497

  • What is the current behavior? (You can also link to an open issue here)

During race condition it is possible that the routerOutlet will activate a component multiple times.

  • What is the new behavior (if this is a feature change)?

Ensure that activation can happen only once.

  • Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

No

  • Other information:

@mhevery mhevery force-pushed the issue7497 branch 2 times, most recently from 33fafd9 to dbe4485 Compare March 11, 2016 04:59
@mhevery
Copy link
Copy Markdown
Contributor Author

mhevery commented Mar 11, 2016

@tbosch this is a fix for #7497. I need help writing the test which demonstrates the issue.

@mhevery
Copy link
Copy Markdown
Contributor Author

mhevery commented Mar 16, 2016

@tbosch LGTM, per our discussion

@jelbourn jelbourn changed the title Issue7497 fix(router): RouterOutlet loads component twice in a race condition Mar 16, 2016
@mary-poppins
Copy link
Copy Markdown

Merging PR #7545 on behalf of @jelbourn to branch presubmit-jelbourn-pr-7545.

@mhevery mhevery closed this in 2f581ff Mar 16, 2016
@mhevery mhevery deleted the issue7497 branch June 2, 2017 17:04
@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 11, 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.

RouterOutlet loads component 2x in a race condition

5 participants

X Tutup