X Tutup
Skip to content

fix(injectors): reset the construction counter in dynamic strategy.#3635

Closed
rkirov wants to merge 1 commit intoangular:masterfrom
rkirov:dehydrate_queries
Closed

fix(injectors): reset the construction counter in dynamic strategy.#3635
rkirov wants to merge 1 commit intoangular:masterfrom
rkirov:dehydrate_queries

Conversation

@rkirov
Copy link
Copy Markdown
Contributor

@rkirov rkirov commented Aug 14, 2015

Adds tests for hydrate / dehydrate in cycle.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

a random typo I caught, not related to the rest of the bug fix.

@rkirov rkirov force-pushed the dehydrate_queries branch 2 times, most recently from d928c28 to 5fe0f4f Compare August 14, 2015 04:33
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Did you add this check because some test failed? Maybe add a unit test to make sure that it works when nestedView is not present.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

done.

@vsavkin vsavkin assigned rkirov and unassigned vsavkin Aug 17, 2015
Adds tests for hydrate / dehydrate in cycle.
@rkirov rkirov force-pushed the dehydrate_queries branch from 5fe0f4f to 815b06c Compare August 18, 2015 18:36
@mary-poppins
Copy link
Copy Markdown

Merging PR #3635 on behalf of @rkirov to branch presubmit-rkirov-pr-3635.

@rkirov rkirov added the action: merge The PR is ready for merge by the caretaker label Aug 18, 2015
@mary-poppins mary-poppins removed the action: merge The PR is ready for merge by the caretaker label Aug 18, 2015
@rkirov rkirov closed this in 272ad61 Aug 18, 2015
@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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

X Tutup