| route-pattern | /_view_fragments/voltron/pull_requests/show/:user_id/:repository/:id/pull_request_layout(.:format) |
| route-controller | voltron_pull_requests_fragments |
| route-action | pull_request_layout |
| fetch-nonce | v2:42b20d89-e9d0-851d-61f9-f9a2943ec6d5 |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | BD78:193CDB:C73827:105DDFF:696AFC28 |
| html-safe-nonce | 00aee9f3755a990151e3c686fed5dd1b89989401389a597849012c966530ffd9 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJCRDc4OjE5M0NEQjpDNzM4Mjc6MTA1RERGRjo2OTZBRkMyOCIsInZpc2l0b3JfaWQiOiI4NTY5NjkyMTkzODU5MjM4OTUyIiwicmVnaW9uX2VkZ2UiOiJpYWQiLCJyZWdpb25fcmVuZGVyIjoiaWFkIn0= |
| visitor-hmac | c55aa982c5e2f895ecd40c7b3cbb5622e5c92f0ee441d57cc3aeb45f2f3b6465 |
| hovercard-subject-tag | pull_request:219592627 |
| github-keyboard-shortcuts | repository,pull-request-list,pull-request-conversation,pull-request-files-changed,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | ///voltron/pull_requests_fragments/pull_request_layout |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/_view_fragments/voltron/pull_requests/show/angular/angular/26209/pull_request_layout |
| twitter:image | https://opengraph.githubassets.com/c3746beb68076e49458d81839fbc81c589e21a701f2bb64de30e3399b8198912/angular/angular/pull/26209 |
| twitter:card | summary_large_image |
| og:image | https://opengraph.githubassets.com/c3746beb68076e49458d81839fbc81c589e21a701f2bb64de30e3399b8198912/angular/angular/pull/26209 |
| og:image:alt | Fixes #26208
PR Type
[x] Bugfix
What is the current behavior?
Upgraded AngularJS dom elements are not destroyed properly (via jQuery) causing events+data to not be cleaned up. Child elements do no... |
| og:image:width | 1200 |
| og:image:height | 600 |
| og:site_name | GitHub |
| og:type | object |
| og:author:username | jbedard |
| hostname | github.com |
| expected-hostname | github.com |
| None | 5f99f7c1d70f01da5b93e5ca90303359738944d8ab470e396496262c66e60b8d |
| turbo-cache-control | no-preview |
| go-import | github.com/angular/angular git https://github.com/angular/angular.git |
| octolytics-dimension-user_id | 139426 |
| octolytics-dimension-user_login | angular |
| octolytics-dimension-repository_id | 24195339 |
| octolytics-dimension-repository_nwo | angular/angular |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 24195339 |
| octolytics-dimension-repository_network_root_nwo | angular/angular |
| turbo-body-classes | logged-out env-production page-responsive |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 82560a55c6b2054555076f46e683151ee28a19bc |
| ui-target | canary-1 |
| theme-color | #1e2327 |
| color-scheme | light dark |
| Skip to content | https://github.com/angular/angular/pull/26209#start-of-content |
|
| https://github.com/ |
|
Sign in
| https://github.com/login?return_to=https%3A%2F%2Fgithub.com%2Fangular%2Fangular%2Fpull%2F26209 |
| GitHub CopilotWrite better code with AI | https://github.com/features/copilot |
| GitHub SparkBuild and deploy intelligent apps | https://github.com/features/spark |
| GitHub ModelsManage and compare prompts | https://github.com/features/models |
| MCP RegistryNewIntegrate external tools | https://github.com/mcp |
| ActionsAutomate any workflow | https://github.com/features/actions |
| CodespacesInstant dev environments | https://github.com/features/codespaces |
| IssuesPlan and track work | https://github.com/features/issues |
| Code ReviewManage code changes | https://github.com/features/code-review |
| GitHub Advanced SecurityFind and fix vulnerabilities | https://github.com/security/advanced-security |
| Code securitySecure your code as you build | https://github.com/security/advanced-security/code-security |
| Secret protectionStop leaks before they start | https://github.com/security/advanced-security/secret-protection |
| Why GitHub | https://github.com/why-github |
| Documentation | https://docs.github.com |
| Blog | https://github.blog |
| Changelog | https://github.blog/changelog |
| Marketplace | https://github.com/marketplace |
| View all features | https://github.com/features |
| Enterprises | https://github.com/enterprise |
| Small and medium teams | https://github.com/team |
| Startups | https://github.com/enterprise/startups |
| Nonprofits | https://github.com/solutions/industry/nonprofits |
| App Modernization | https://github.com/solutions/use-case/app-modernization |
| DevSecOps | https://github.com/solutions/use-case/devsecops |
| DevOps | https://github.com/solutions/use-case/devops |
| CI/CD | https://github.com/solutions/use-case/ci-cd |
| View all use cases | https://github.com/solutions/use-case |
| Healthcare | https://github.com/solutions/industry/healthcare |
| Financial services | https://github.com/solutions/industry/financial-services |
| Manufacturing | https://github.com/solutions/industry/manufacturing |
| Government | https://github.com/solutions/industry/government |
| View all industries | https://github.com/solutions/industry |
| View all solutions | https://github.com/solutions |
| AI | https://github.com/resources/articles?topic=ai |
| Software Development | https://github.com/resources/articles?topic=software-development |
| DevOps | https://github.com/resources/articles?topic=devops |
| Security | https://github.com/resources/articles?topic=security |
| View all topics | https://github.com/resources/articles |
| Customer stories | https://github.com/customer-stories |
| Events & webinars | https://github.com/resources/events |
| Ebooks & reports | https://github.com/resources/whitepapers |
| Business insights | https://github.com/solutions/executive-insights |
| GitHub Skills | https://skills.github.com |
| Documentation | https://docs.github.com |
| Customer support | https://support.github.com |
| Community forum | https://github.com/orgs/community/discussions |
| Trust center | https://github.com/trust-center |
| Partners | https://github.com/partners |
| GitHub SponsorsFund open source developers | https://github.com/sponsors |
| Security Lab | https://securitylab.github.com |
| Maintainer Community | https://maintainers.github.com |
| Accelerator | https://github.com/accelerator |
| Archive Program | https://archiveprogram.github.com |
| Topics | https://github.com/topics |
| Trending | https://github.com/trending |
| Collections | https://github.com/collections |
| Enterprise platformAI-powered developer platform | https://github.com/enterprise |
| GitHub Advanced SecurityEnterprise-grade security features | https://github.com/security/advanced-security |
| Copilot for BusinessEnterprise-grade AI features | https://github.com/features/copilot/copilot-business |
| Premium SupportEnterprise-grade 24/7 support | https://github.com/premium-support |
| Pricing | https://github.com/pricing |
| Search syntax tips | https://docs.github.com/search-github/github-code-search/understanding-github-code-search-syntax |
| documentation | https://docs.github.com/search-github/github-code-search/understanding-github-code-search-syntax |
|
Sign in
| https://github.com/login?return_to=https%3A%2F%2Fgithub.com%2Fangular%2Fangular%2Fpull%2F26209 |
|
Sign up
| https://github.com/signup?ref_cta=Sign+up&ref_loc=header+logged+out&ref_page=%2F%3Cuser-name%3E%2F%3Crepo-name%3E%2Fvoltron%2Fpull_requests_fragments%2Fpull_request_layout&source=header-repo&source_repo=angular%2Fangular |
| Reload | https://github.com/angular/angular/pull/26209 |
| Reload | https://github.com/angular/angular/pull/26209 |
| Reload | https://github.com/angular/angular/pull/26209 |
|
angular
| https://github.com/angular |
| angular | https://github.com/angular/angular |
|
Notifications
| https://github.com/login?return_to=%2Fangular%2Fangular |
|
Fork
27k
| https://github.com/login?return_to=%2Fangular%2Fangular |
|
Star
99.7k
| https://github.com/login?return_to=%2Fangular%2Fangular |
|
Code
| https://github.com/angular/angular |
|
Issues
1k
| https://github.com/angular/angular/issues |
|
Pull requests
104
| https://github.com/angular/angular/pulls |
|
Discussions
| https://github.com/angular/angular/discussions |
|
Actions
| https://github.com/angular/angular/actions |
|
Projects
2
| https://github.com/angular/angular/projects |
|
Security
Uh oh!
There was an error while loading. Please reload this page.
| https://github.com/angular/angular/security |
| Please reload this page | https://github.com/angular/angular/pull/26209 |
|
Insights
| https://github.com/angular/angular/pulse |
|
Code
| https://github.com/angular/angular |
|
Issues
| https://github.com/angular/angular/issues |
|
Pull requests
| https://github.com/angular/angular/pulls |
|
Discussions
| https://github.com/angular/angular/discussions |
|
Actions
| https://github.com/angular/angular/actions |
|
Projects
| https://github.com/angular/angular/projects |
|
Security
| https://github.com/angular/angular/security |
|
Insights
| https://github.com/angular/angular/pulse |
| Sign up for GitHub
| https://github.com/signup?return_to=%2Fangular%2Fangular%2Fissues%2Fnew%2Fchoose |
| terms of service | https://docs.github.com/terms |
| privacy statement | https://docs.github.com/privacy |
| Sign in | https://github.com/login?return_to=%2Fangular%2Fangular%2Fissues%2Fnew%2Fchoose |
| Jump to bottom | https://github.com/angular/angular/pull/26209#issue-comment-box |
| jbedard | https://github.com/jbedard |
| angular:master | https://github.com/angular/angular/tree/master |
| jbedard:upgraded-cleandata | https://github.com/jbedard/angular/tree/upgraded-cleandata |
|
fix(upgrade): properly destroy upgraded component elements and descendants
| https://github.com/angular/angular/pull/26209#top |
| jbedard | https://github.com/jbedard |
| angular:master | https://github.com/angular/angular/tree/master |
| jbedard:upgraded-cleandata | https://github.com/jbedard/angular/tree/upgraded-cleandata |
|
Conversation
47
| https://github.com/angular/angular/pull/26209 |
|
Commits
4
| https://github.com/angular/angular/pull/26209/commits |
|
Checks
0
| https://github.com/angular/angular/pull/26209/checks |
|
Files changed
| https://github.com/angular/angular/pull/26209/files |
| Please reload this page | https://github.com/angular/angular/pull/26209 |
| https://github.co/hiddenchars |
| https://github.com/angular/angular/pull/{{ revealButtonHref }} |
|
| https://github.com/jbedard |
| jbedard | https://github.com/jbedard |
| Oct 2, 2018 | https://github.com/angular/angular/pull/26209#issue-365750717 |
| #26208 | https://github.com/angular/angular/issues/26208 |
| #26208 | https://github.com/angular/angular/issues/26208 |
| Please reload this page | https://github.com/angular/angular/pull/26209 |
| https://github.com/googlebot |
| googlebot | https://github.com/googlebot |
|
cla: yes
| https://github.com/angular/angular/issues?q=state%3Aopen%20label%3A%22cla%3A%20yes%22 |
| Oct 2, 2018 | https://github.com/angular/angular/pull/26209#event-1878425761 |
| https://github.com/jbedard |
| jbedard | https://github.com/jbedard |
| force-pushed | https://github.com/angular/angular/compare/53267b42c9cabae02f2db9887999b42594155c01..528fe26240eb94b69a27ea0146679e16f83d472d |
| 53267b4 | https://github.com/angular/angular/commit/53267b42c9cabae02f2db9887999b42594155c01 |
| 528fe26 | https://github.com/angular/angular/commit/528fe26240eb94b69a27ea0146679e16f83d472d |
|
Compare
| https://github.com/angular/angular/compare/53267b42c9cabae02f2db9887999b42594155c01..528fe26240eb94b69a27ea0146679e16f83d472d |
| October 2, 2018 06:28 | https://github.com/angular/angular/pull/26209#event-1878464715 |
| https://github.com/gkalpak |
| gkalpak | https://github.com/gkalpak |
|
type: bug/fix
| https://github.com/angular/angular/issues?q=state%3Aopen%20label%3A%22type%3A%20bug%2Ffix%22 |
|
action: review
| https://github.com/angular/angular/issues?q=state%3Aopen%20label%3A%22action%3A%20review%22 |
|
comp: upgrade/static
| https://github.com/angular/angular/issues?q=state%3Aopen%20label%3A%22comp%3A%20upgrade%2Fstatic%22 |
|
target: patch
| https://github.com/angular/angular/issues?q=state%3Aopen%20label%3A%22target%3A%20patch%22 |
| Oct 2, 2018 | https://github.com/angular/angular/pull/26209#event-1878897118 |
| https://github.com/gkalpak |
| gkalpak | https://github.com/gkalpak |
|
Oct 2, 2018
| https://github.com/angular/angular/pull/26209#pullrequestreview-160652868 |
|
View reviewed changes
| https://github.com/angular/angular/pull/26209/files |
| gkalpak | https://github.com/gkalpak |
| https://github.com/angular/angular/pull/26209#pullrequestreview-160652868 |
| Learn more | https://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment |
| Please reload this page | https://github.com/angular/angular/pull/26209 |
| packages/upgrade/src/common/angular1.ts | https://github.com/angular/angular/pull/26209/files#diff-f4bc608dcf49f1570178492bb5a4a118aa82dfe2546ceb9919a1431e22805803 |
| gkalpak | https://github.com/gkalpak |
| Oct 2, 2018 | https://github.com/angular/angular/pull/26209#discussion_r221896309 |
| Learn more | https://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment |
| Please reload this page | https://github.com/angular/angular/pull/26209 |
| jbedard | https://github.com/jbedard |
| Oct 2, 2018 | https://github.com/angular/angular/pull/26209#discussion_r222029617 |
| Learn more | https://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment |
| Please reload this page | https://github.com/angular/angular/pull/26209 |
| jbedard | https://github.com/jbedard |
| Oct 2, 2018 | https://github.com/angular/angular/pull/26209#discussion_r222030318 |
| Learn more | https://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment |
| Please reload this page | https://github.com/angular/angular/pull/26209 |
| gkalpak | https://github.com/gkalpak |
| Oct 2, 2018 | https://github.com/angular/angular/pull/26209#discussion_r222041684 |
| Learn more | https://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment |
| Please reload this page | https://github.com/angular/angular/pull/26209 |
| jbedard | https://github.com/jbedard |
| Oct 2, 2018 | https://github.com/angular/angular/pull/26209#discussion_r222044818 |
| Learn more | https://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment |
| Please reload this page | https://github.com/angular/angular/pull/26209 |
| gkalpak | https://github.com/gkalpak |
| Oct 3, 2018 | https://github.com/angular/angular/pull/26209#discussion_r222262843 |
| Learn more | https://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment |
| Please reload this page | https://github.com/angular/angular/pull/26209 |
| packages/upgrade/src/common/upgrade_helper.ts | https://github.com/angular/angular/pull/26209/files#diff-3611fb7a0f6bd5f6a1232eb1c4ef96614548842996994fdd9bc111a682c53592 |
| gkalpak | https://github.com/gkalpak |
| Oct 2, 2018 | https://github.com/angular/angular/pull/26209#discussion_r221898019 |
| Learn more | https://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment |
| AngularJS implementation | https://github.com/angular/angular.js/blob/26ddc5f830f902a3d22f4b2aab70d86d4d688c82/src/jqLite.js#L306-L312 |
| jsLiteAcceptsData() | https://github.com/angular/angular.js/blob/26ddc5f830f902a3d22f4b2aab70d86d4d688c82/src/jqLite.js#L307 |
| if (element.querySelectorAll) | https://github.com/angular/angular.js/blob/26ddc5f830f902a3d22f4b2aab70d86d4d688c82/src/jqLite.js#L309 |
| Please reload this page | https://github.com/angular/angular/pull/26209 |
| jbedard | https://github.com/jbedard |
| Oct 2, 2018 | https://github.com/angular/angular/pull/26209#discussion_r222049232 |
| Learn more | https://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment |
| acceptsData | https://github.com/jquery/jquery/blob/58c6ca9822afa42d3b40cca8edb0abe90a2bcb34/src/data/var/acceptData.js#L10-L16 |
| Please reload this page | https://github.com/angular/angular/pull/26209 |
| jbedard | https://github.com/jbedard |
| Oct 2, 2018 | https://github.com/angular/angular/pull/26209#discussion_r222051192 |
| Learn more | https://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment |
| Please reload this page | https://github.com/angular/angular/pull/26209 |
| jbedard | https://github.com/jbedard |
| Oct 3, 2018 | https://github.com/angular/angular/pull/26209#discussion_r222477676 |
| Learn more | https://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment |
| jqLite | https://github.com/angular/angular.js/blob/26ddc5f830f902a3d22f4b2aab70d86d4d688c82/src/jqLite.js#L306-L312 |
| jQuery | https://github.com/jquery/jquery/blob/e743cbd28553267f955f71ea7248377915613fd9/src/manipulation.js#L223 |
| Please reload this page | https://github.com/angular/angular/pull/26209 |
| gkalpak | https://github.com/gkalpak |
| Oct 4, 2018 | https://github.com/angular/angular/pull/26209#discussion_r222567617 |
| Learn more | https://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment |
| Please reload this page | https://github.com/angular/angular/pull/26209 |
| packages/upgrade/test/static/integration/upgrade_component_spec.ts | https://github.com/angular/angular/pull/26209/files#diff-138f4e83e4db39f790084bbee8e6f49f21af7f72ba7683739d686bd7b95473e4 |
| gkalpak | https://github.com/gkalpak |
| Oct 2, 2018 | https://github.com/angular/angular/pull/26209#discussion_r221898778 |
| Learn more | https://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment |
| Please reload this page | https://github.com/angular/angular/pull/26209 |
| https://github.com/jbedard |
| jbedard | https://github.com/jbedard |
| Oct 3, 2018 | https://github.com/angular/angular/pull/26209#issuecomment-426819796 |
| Please reload this page | https://github.com/angular/angular/pull/26209 |
| https://github.com/gkalpak |
| gkalpak | https://github.com/gkalpak |
|
Oct 4, 2018
| https://github.com/angular/angular/pull/26209#pullrequestreview-161483741 |
|
View reviewed changes
| https://github.com/angular/angular/pull/26209/files |
| gkalpak | https://github.com/gkalpak |
| https://github.com/angular/angular/pull/26209#pullrequestreview-161483741 |
| Learn more | https://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment |
| under certain circumstances | https://github.com/angular/angular/blob/aaaa34021c2d56f798d20e5a1f31b23972055170/packages/upgrade/src/dynamic/upgrade_ng1_adapter.ts#L168-L171 |
| Please reload this page | https://github.com/angular/angular/pull/26209 |
| packages/upgrade/src/common/angular1.ts | https://github.com/angular/angular/pull/26209/files#diff-f4bc608dcf49f1570178492bb5a4a118aa82dfe2546ceb9919a1431e22805803 |
| gkalpak | https://github.com/gkalpak |
| Oct 4, 2018 | https://github.com/angular/angular/pull/26209#discussion_r222567958 |
| Learn more | https://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment |
| Please reload this page | https://github.com/angular/angular/pull/26209 |
| packages/upgrade/test/dynamic/upgrade_spec.ts | https://github.com/angular/angular/pull/26209/files#diff-2d4d0d23f889701902fc796e618c0f8d26f98ce3939253866b82eced4b82fff9 |
| gkalpak | https://github.com/gkalpak |
| Oct 4, 2018 | https://github.com/angular/angular/pull/26209#discussion_r222568701 |
| Learn more | https://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment |
| Please reload this page | https://github.com/angular/angular/pull/26209 |
| https://github.com/jbedard |
| jbedard | https://github.com/jbedard |
| force-pushed | https://github.com/angular/angular/compare/d32c85c4f2b9892b4db34589874eaf24280010c7..f9b55a62ec07fa34f65a89321c4435fb83ef3a0e |
| d32c85c | https://github.com/angular/angular/commit/d32c85c4f2b9892b4db34589874eaf24280010c7 |
| f9b55a6 | https://github.com/angular/angular/commit/f9b55a62ec07fa34f65a89321c4435fb83ef3a0e |
|
Compare
| https://github.com/angular/angular/compare/d32c85c4f2b9892b4db34589874eaf24280010c7..f9b55a62ec07fa34f65a89321c4435fb83ef3a0e |
| October 5, 2018 08:20 | https://github.com/angular/angular/pull/26209#event-1886517089 |
| https://github.com/gkalpak |
| gkalpak | https://github.com/gkalpak |
|
Oct 5, 2018
| https://github.com/angular/angular/pull/26209#pullrequestreview-161972574 |
|
View reviewed changes
| https://github.com/angular/angular/pull/26209/files |
| gkalpak | https://github.com/gkalpak |
| https://github.com/angular/angular/pull/26209#pullrequestreview-161972574 |
| Learn more | https://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment |
| Please reload this page | https://github.com/angular/angular/pull/26209 |
| packages/upgrade/src/common/upgrade_helper.ts | https://github.com/angular/angular/pull/26209/files#diff-3611fb7a0f6bd5f6a1232eb1c4ef96614548842996994fdd9bc111a682c53592 |
| Please reload this page | https://github.com/angular/angular/pull/26209 |
| packages/upgrade/test/dynamic/upgrade_spec.ts | https://github.com/angular/angular/pull/26209/files#diff-2d4d0d23f889701902fc796e618c0f8d26f98ce3939253866b82eced4b82fff9 |
| gkalpak | https://github.com/gkalpak |
| Oct 5, 2018 | https://github.com/angular/angular/pull/26209#discussion_r222958629 |
| Learn more | https://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment |
| Please reload this page | https://github.com/angular/angular/pull/26209 |
| https://github.com/gkalpak |
| gkalpak | https://github.com/gkalpak |
|
action: cleanup
| https://github.com/angular/angular/issues?q=state%3Aopen%20label%3A%22action%3A%20cleanup%22 |
|
action: review
| https://github.com/angular/angular/issues?q=state%3Aopen%20label%3A%22action%3A%20review%22 |
| Oct 5, 2018 | https://github.com/angular/angular/pull/26209#event-1886785120 |
| https://github.com/jbedard |
| jbedard | https://github.com/jbedard |
| force-pushed | https://github.com/angular/angular/compare/f9b55a62ec07fa34f65a89321c4435fb83ef3a0e..e383c49bb9f429e3fb31b8d265f9e1a0375cd8ae |
| f9b55a6 | https://github.com/angular/angular/commit/f9b55a62ec07fa34f65a89321c4435fb83ef3a0e |
| e383c49 | https://github.com/angular/angular/commit/e383c49bb9f429e3fb31b8d265f9e1a0375cd8ae |
|
Compare
| https://github.com/angular/angular/compare/f9b55a62ec07fa34f65a89321c4435fb83ef3a0e..e383c49bb9f429e3fb31b8d265f9e1a0375cd8ae |
| October 5, 2018 17:07 | https://github.com/angular/angular/pull/26209#event-1887662299 |
|
| https://github.com/jbedard |
| fix(upgrade): properly destroy upgraded component elements and descen… | https://github.com/angular/angular/pull/26209/commits/10bf47eb443d3f79a55855f5e2d39cf67d97d29e |
| 10bf47e | https://github.com/angular/angular/pull/26209/commits/10bf47eb443d3f79a55855f5e2d39cf67d97d29e |
| angular#26208 | https://github.com/angular/angular/issues/26208 |
| https://github.com/jbedard |
| jbedard | https://github.com/jbedard |
| force-pushed | https://github.com/angular/angular/compare/e383c49bb9f429e3fb31b8d265f9e1a0375cd8ae..10bf47eb443d3f79a55855f5e2d39cf67d97d29e |
| e383c49 | https://github.com/angular/angular/commit/e383c49bb9f429e3fb31b8d265f9e1a0375cd8ae |
| 10bf47e | https://github.com/angular/angular/commit/10bf47eb443d3f79a55855f5e2d39cf67d97d29e |
|
Compare
| https://github.com/angular/angular/compare/e383c49bb9f429e3fb31b8d265f9e1a0375cd8ae..10bf47eb443d3f79a55855f5e2d39cf67d97d29e |
| October 5, 2018 17:42 | https://github.com/angular/angular/pull/26209#event-1887730853 |
| https://github.com/jbedard |
| jbedard | https://github.com/jbedard |
| Oct 5, 2018 | https://github.com/angular/angular/pull/26209#issuecomment-427455670 |
| Please reload this page | https://github.com/angular/angular/pull/26209 |
| https://github.com/gkalpak |
| gkalpak | https://github.com/gkalpak |
|
action: merge
| https://github.com/angular/angular/issues?q=state%3Aopen%20label%3A%22action%3A%20merge%22 |
|
merge: caretaker note
| https://github.com/angular/angular/issues?q=state%3Aopen%20label%3A%22merge%3A%20caretaker%20note%22 |
|
action: cleanup
| https://github.com/angular/angular/issues?q=state%3Aopen%20label%3A%22action%3A%20cleanup%22 |
| Oct 6, 2018 | https://github.com/angular/angular/pull/26209#event-1888492400 |
| https://github.com/gkalpak |
| gkalpak | https://github.com/gkalpak |
| Oct 6, 2018 | https://github.com/angular/angular/pull/26209#issuecomment-427558848 |
| @jbedard | https://github.com/jbedard |
| Please reload this page | https://github.com/angular/angular/pull/26209 |
| https://github.com/jasonaden |
| jasonaden | https://github.com/jasonaden |
| Oct 8, 2018 | https://github.com/angular/angular/pull/26209#issuecomment-427926197 |
| Presubmit | http://test/OCL:216214304:BASE:216214360:1539021616695:3a8b4574 |
| Please reload this page | https://github.com/angular/angular/pull/26209 |
| jasonaden | https://github.com/jasonaden |
|
Oct 8, 2018
| https://github.com/angular/angular/pull/26209#ref-commit-623adbb |
|
| https://github.com/jbedard |
|
| https://github.com/jasonaden |
| fix(upgrade): properly destroy upgraded component elements and descen… | https://github.com/angular/angular/commit/623adbbdf7c052f8f493bd79f863f4647bfa6cde |
| 623adbb | https://github.com/angular/angular/commit/623adbbdf7c052f8f493bd79f863f4647bfa6cde |
| #26209 | https://github.com/angular/angular/pull/26209 |
| #26208 | https://github.com/angular/angular/issues/26208 |
| #26209 | https://github.com/angular/angular/pull/26209 |
| https://github.com/jasonaden |
| jasonaden | https://github.com/jasonaden |
| 5a31bde | https://github.com/angular/angular/commit/5a31bde6496c63d82d86227d7772c2127c28b08c |
| Oct 8, 2018 | https://github.com/angular/angular/pull/26209#event-1891085142 |
| jasonaden | https://github.com/jasonaden |
|
Oct 8, 2018
| https://github.com/angular/angular/pull/26209#ref-commit-9623e7c |
|
| https://github.com/jasonaden |
| Revert "fix(upgrade): properly destroy upgraded component elements an… | https://github.com/jasonaden/angular/commit/9623e7c639bd30bb03869b9e62a64cf86969ce3e |
| 9623e7c | https://github.com/jasonaden/angular/commit/9623e7c639bd30bb03869b9e62a64cf86969ce3e |
| angular#26209 | https://github.com/angular/angular/pull/26209 |
| 5a31bde | https://github.com/jasonaden/angular/commit/5a31bde6496c63d82d86227d7772c2127c28b08c |
| jasonaden | https://github.com/jasonaden |
|
Oct 8, 2018
| https://github.com/angular/angular/pull/26209#ref-commit-b22c376 |
|
| https://github.com/jasonaden |
| Revert "fix(upgrade): properly destroy upgraded component elements an… | https://github.com/angular/angular/commit/b22c3761234c153ffe160c45f6a0970b3efee599 |
| b22c376 | https://github.com/angular/angular/commit/b22c3761234c153ffe160c45f6a0970b3efee599 |
| #26209 | https://github.com/angular/angular/pull/26209 |
| 623adbb | https://github.com/angular/angular/commit/623adbbdf7c052f8f493bd79f863f4647bfa6cde |
| https://github.com/jasonaden |
| jasonaden | https://github.com/jasonaden |
| Oct 8, 2018 | https://github.com/angular/angular/pull/26209#event-1891355360 |
| https://github.com/jasonaden |
| jasonaden | https://github.com/jasonaden |
| Oct 8, 2018 | https://github.com/angular/angular/pull/26209#issuecomment-427988380 |
| @gkalpak | https://github.com/gkalpak |
| @jbedard | https://github.com/jbedard |
| Please reload this page | https://github.com/angular/angular/pull/26209 |
| https://github.com/jasonaden |
| jasonaden | https://github.com/jasonaden |
|
action: cleanup
| https://github.com/angular/angular/issues?q=state%3Aopen%20label%3A%22action%3A%20cleanup%22 |
| Oct 8, 2018 | https://github.com/angular/angular/pull/26209#event-1891362280 |
| jasonaden | https://github.com/jasonaden |
|
Oct 10, 2018
| https://github.com/angular/angular/pull/26209#ref-commit-7466a99 |
|
| https://github.com/jasonaden |
| Revert "fix(upgrade): properly destroy upgraded component elements an… | https://github.com/jasonaden/angular/commit/7466a99dda68cbfada4bc986a9bca80b87e05702 |
| 7466a99 | https://github.com/jasonaden/angular/commit/7466a99dda68cbfada4bc986a9bca80b87e05702 |
| angular#26209 | https://github.com/angular/angular/pull/26209 |
| 912f3d1 | https://github.com/jasonaden/angular/commit/912f3d186f4ce5c4a97268775509cfb6285916c2 |
| jasonaden | https://github.com/jasonaden |
|
Oct 10, 2018
| https://github.com/angular/angular/pull/26209#ref-commit-65555fe |
|
| https://github.com/jasonaden |
| Revert "fix(upgrade): properly destroy upgraded component elements an… | https://github.com/angular/angular/commit/65555fe35d9ae3e073c8d293923a0c21e742e019 |
| 65555fe | https://github.com/angular/angular/commit/65555fe35d9ae3e073c8d293923a0c21e742e019 |
| #26209 | https://github.com/angular/angular/pull/26209 |
| 6da3867 | https://github.com/angular/angular/commit/6da3867d638baa71eea8f3e3741e0afccc98e86b |
| https://github.com/jasonaden |
| jasonaden | https://github.com/jasonaden |
| Oct 10, 2018 | https://github.com/angular/angular/pull/26209#event-1896732288 |
| https://github.com/jasonaden |
| jasonaden | https://github.com/jasonaden |
| Oct 10, 2018 | https://github.com/angular/angular/pull/26209#issuecomment-428746664 |
| Please reload this page | https://github.com/angular/angular/pull/26209 |
| https://github.com/jasonaden |
| jasonaden | https://github.com/jasonaden |
| Oct 10, 2018 | https://github.com/angular/angular/pull/26209#issuecomment-428747513 |
| https://travis-ci.org/angular/angular/jobs/439854930 | https://travis-ci.org/angular/angular/jobs/439854930 |
| Please reload this page | https://github.com/angular/angular/pull/26209 |
| https://github.com/jbedard |
| jbedard | https://github.com/jbedard |
| Oct 10, 2018 | https://github.com/angular/angular/pull/26209#issuecomment-428750194 |
| @gkalpak | https://github.com/gkalpak |
| f5447a3 | https://github.com/angular/angular/commit/f5447a315d7189867bcd20e0eaaaba6d9d54b0b8 |
| Please reload this page | https://github.com/angular/angular/pull/26209 |
| https://github.com/gkalpak |
| gkalpak | https://github.com/gkalpak |
| Oct 10, 2018 | https://github.com/angular/angular/pull/26209#issuecomment-428752230 |
| Please reload this page | https://github.com/angular/angular/pull/26209 |
|
| https://github.com/gkalpak |
| fixup! fix(upgrade): properly destroy upgraded component elements and… | https://github.com/angular/angular/pull/26209/commits/3586fc6bfaa043cc8ee59fc07b4ecc9d8c76a8ff |
| 3586fc6 | https://github.com/angular/angular/pull/26209/commits/3586fc6bfaa043cc8ee59fc07b4ecc9d8c76a8ff |
| https://github.com/googlebot |
| googlebot | https://github.com/googlebot |
| Oct 10, 2018 | https://github.com/angular/angular/pull/26209#issuecomment-428754028 |
| Please reload this page | https://github.com/angular/angular/pull/26209 |
| https://github.com/googlebot |
| googlebot | https://github.com/googlebot |
|
cla: no
| https://github.com/angular/angular/issues?q=state%3Aopen%20label%3A%22cla%3A%20no%22 |
|
cla: yes
| https://github.com/angular/angular/issues?q=state%3Aopen%20label%3A%22cla%3A%20yes%22 |
| Oct 10, 2018 | https://github.com/angular/angular/pull/26209#event-1896791812 |
| https://github.com/gkalpak |
| gkalpak | https://github.com/gkalpak |
| Oct 11, 2018 | https://github.com/angular/angular/pull/26209#issuecomment-428855004 |
| Please reload this page | https://github.com/angular/angular/pull/26209 |
| https://github.com/mhevery |
| mhevery | https://github.com/mhevery |
|
cla: yes
| https://github.com/angular/angular/issues?q=state%3Aopen%20label%3A%22cla%3A%20yes%22 |
|
cla: no
| https://github.com/angular/angular/issues?q=state%3Aopen%20label%3A%22cla%3A%20no%22 |
| Oct 11, 2018 | https://github.com/angular/angular/pull/26209#event-1899160074 |
| https://github.com/googlebot |
| googlebot | https://github.com/googlebot |
| Oct 11, 2018 | https://github.com/angular/angular/pull/26209#issuecomment-429093571 |
| Please reload this page | https://github.com/angular/angular/pull/26209 |
| https://github.com/mhevery |
| mhevery | https://github.com/mhevery |
| Oct 11, 2018 | https://github.com/angular/angular/pull/26209#issuecomment-429121042 |
| http://test/OCL:216758713:BASE:216758772:1539292493195:9b1e806b | http://test/OCL:216758713:BASE:216758772:1539292493195:9b1e806b |
| Please reload this page | https://github.com/angular/angular/pull/26209 |
| https://github.com/gkalpak |
| gkalpak | https://github.com/gkalpak |
|
Oct 11, 2018
| https://github.com/angular/angular/pull/26209#pullrequestreview-164037388 |
|
View reviewed changes
| https://github.com/angular/angular/pull/26209/files/3586fc6bfaa043cc8ee59fc07b4ecc9d8c76a8ff |
| https://github.com/mhevery |
| mhevery | https://github.com/mhevery |
|
merge: caretaker note
| https://github.com/angular/angular/issues?q=state%3Aopen%20label%3A%22merge%3A%20caretaker%20note%22 |
| Oct 11, 2018 | https://github.com/angular/angular/pull/26209#event-1899510154 |
| mhevery | https://github.com/mhevery |
|
Oct 12, 2018
| https://github.com/angular/angular/pull/26209#ref-commit-4f09f7d |
|
| https://github.com/jbedard |
|
| https://github.com/mhevery |
| fix(upgrade): properly destroy upgraded component elements and descen… | https://github.com/angular/angular/commit/4f09f7db732eac93e90fe50072509225c4fe6355 |
| 4f09f7d | https://github.com/angular/angular/commit/4f09f7db732eac93e90fe50072509225c4fe6355 |
| #26209 | https://github.com/angular/angular/pull/26209 |
| #26208 | https://github.com/angular/angular/issues/26208 |
| #26209 | https://github.com/angular/angular/pull/26209 |
| https://github.com/mhevery |
| mhevery | https://github.com/mhevery |
| 071934e | https://github.com/angular/angular/commit/071934e92ab638b7b69374bbe1a19109c0353b97 |
| Oct 12, 2018 | https://github.com/angular/angular/pull/26209#event-1899827687 |
| https://github.com/gkalpak |
| gkalpak | https://github.com/gkalpak |
| Oct 12, 2018 | https://github.com/angular/angular/pull/26209#issuecomment-429224534 |
| Please reload this page | https://github.com/angular/angular/pull/26209 |
| https://github.com/apps/angular-automatic-lock-bot |
| angular-automatic-lock-bot | https://github.com/apps/angular-automatic-lock-bot |
| Sep 14, 2019 | https://github.com/angular/angular/pull/26209#issuecomment-531445731 |
| automatic conversation locking policy | https://github.com/angular/angular/blob/67d80f/docs/GITHUB_PROCESS.md#conversation-locking |
| Please reload this page | https://github.com/angular/angular/pull/26209 |
| https://github.com/apps/angular-automatic-lock-bot |
| angular-automatic-lock-bot | https://github.com/apps/angular-automatic-lock-bot |
| Sep 14, 2019 | https://github.com/angular/angular/pull/26209#event-2633926940 |
| Sign up for free | https://github.com/join?source=comment-repo |
| Sign in | https://github.com/login?return_to=https%3A%2F%2Fgithub.com%2Fangular%2Fangular%2Fpull%2F26209 |
|
| https://github.com/gkalpak |
|
gkalpak
| https://github.com/gkalpak |
|
| https://github.com/angular/angular/pull/26209/files/3586fc6bfaa043cc8ee59fc07b4ecc9d8c76a8ff |
|
action: merge
| https://github.com/angular/angular/issues?q=state%3Aopen%20label%3A%22action%3A%20merge%22 |
|
cla: yes
| https://github.com/angular/angular/issues?q=state%3Aopen%20label%3A%22cla%3A%20yes%22 |
|
target: patch
| https://github.com/angular/angular/issues?q=state%3Aopen%20label%3A%22target%3A%20patch%22 |
|
type: bug/fix
| https://github.com/angular/angular/issues?q=state%3Aopen%20label%3A%22type%3A%20bug%2Ffix%22 |
|
Upgraded element children not destroyed/cleaned
| https://github.com/angular/angular/issues/26208 |
| Please reload this page | https://github.com/angular/angular/pull/26209 |
|
| https://github.com/jbedard |
|
| https://github.com/gkalpak |
|
| https://github.com/jasonaden |
|
| https://github.com/googlebot |
|
| https://github.com/IgorMinar |
|
| https://github.com/mhevery |
|
| https://github.com |
| Terms | https://docs.github.com/site-policy/github-terms/github-terms-of-service |
| Privacy | https://docs.github.com/site-policy/privacy-policies/github-privacy-statement |
| Security | https://github.com/security |
| Status | https://www.githubstatus.com/ |
| Community | https://github.community/ |
| Docs | https://docs.github.com/ |
| Contact | https://support.github.com?tags=dotcom-footer |