Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Check for dataContext to avoid recyclingTemplate when unneeded #1208

Merged
merged 10 commits into from
Aug 2, 2019

Conversation

NicolasChampagne
Copy link
Contributor

PR Type

What kind of change does this PR introduce?
Bugfix

What is the current behavior?

Would recycle template even when unneeded

What is the new behavior?

Won't recycle template even when unneeded

PR Checklist

Please check if your PR fulfills the following requirements:

Internal Issue (If applicable):
https://nventive.visualstudio.com/Umbrella/_workitems/edit/144025

@NicolasChampagne NicolasChampagne force-pushed the dev/nich/template-recycling branch 2 times, most recently from 0d23860 to fbc39b1 Compare July 8, 2019 18:13
@ghuntley ghuntley added platform/android 🤖 Categorizes an issue or PR as relevant to the Android platform platform/ios 🍎 Categorizes an issue or PR as relevant to the iOS platform platform/wasm 🌐 Categorizes an issue or PR as relevant to the WebAssembly platform kind/bug Something isn't working labels Jul 9, 2019
@NicolasChampagne NicolasChampagne force-pushed the dev/nich/template-recycling branch from e03f65d to 6df9fb7 Compare July 15, 2019 20:47
@NicolasChampagne NicolasChampagne force-pushed the dev/nich/template-recycling branch from ac47f55 to 7181a9f Compare July 23, 2019 14:58
@NicolasChampagne NicolasChampagne force-pushed the dev/nich/template-recycling branch from 7181a9f to 6a97a9b Compare July 23, 2019 15:15
@NicolasChampagne NicolasChampagne requested review from jeromelaban and MatFillion and removed request for jeromelaban and MatFillion July 29, 2019 14:22
@NicolasChampagne NicolasChampagne merged commit 5da7c8f into master Aug 2, 2019
@delete-merged-branch delete-merged-branch bot deleted the dev/nich/template-recycling branch August 2, 2019 18:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Something isn't working platform/android 🤖 Categorizes an issue or PR as relevant to the Android platform platform/ios 🍎 Categorizes an issue or PR as relevant to the iOS platform platform/wasm 🌐 Categorizes an issue or PR as relevant to the WebAssembly platform
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants