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

fix: Fixed issues caused by cleaning up .NET objects. #28

Merged
merged 2 commits into from
Jul 11, 2021

Conversation

canhorn
Copy link
Owner

@canhorn canhorn commented Jul 11, 2021

Keeping the JavaScript and .NET objects in sync causes more overhead and management that just trigging a delete during a .NET object finalize call.

Because the .NET object can go out of scope, for some reason or another, it could cause the JavaScript reference to be lost, when the cache is cleared.

To make this library as easy as possible to use without having to micro manage the state of all objects created/destroyed, it will just hold onto the cache for the life of the browser page request.

The automatic cache remove was reverted, the 'removeEntity' interop is still available for custom code to be easily created.

General Styles and Cleanup needed after adding .editorconfig.

Keeping the JavaScript and .NET objects in sync causes more overhead and management that just trigging a delete during a .NET object finalize call.

Because the .NET object can go out of scope, for some reason or another, it could cause the JavaScript reference to be lost, when the cache is cleared.

To make this library as easy as possible to use without having to micro manage the state of all objects created/destroyed, it will just hold onto the cache for the life of the browser page request.

The automatic cache remove was reverted, the 'removeEntity' interop is still available for custom code to be easily created.

General Styles and Cleanup needed after adding .editorconfig.
@github-actions
Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://polite-plant-0f8750a10-28.centralus.azurestaticapps.net

@github-actions
Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://polite-plant-0f8750a10-28.centralus.azurestaticapps.net

@canhorn canhorn merged commit f90b08f into main Jul 11, 2021
@canhorn canhorn deleted the bug/revert_cache_cleanup branch July 11, 2021 17:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant