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

Potential fix for #96 #98

Merged
merged 1 commit into from
Apr 14, 2021
Merged

Potential fix for #96 #98

merged 1 commit into from
Apr 14, 2021

Conversation

withmask
Copy link
Contributor

@withmask withmask commented Apr 7, 2021

This edit doesn't try to overwrite setters/getters as it's completely a new object. Have not had any issues with it so far.

@terehov
Copy link
Contributor

terehov commented Apr 14, 2021

Thank you very much. Looks like a good fix. I'll check it and will let you know 👍

@terehov terehov merged commit 0620eee into fullstack-build:master Apr 14, 2021
@terehov
Copy link
Contributor

terehov commented Apr 14, 2021

Seems to work fine! I created a release, does it fix #96 for you?

https://github.com/fullstack-build/tslog/releases/tag/v3.2.0

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.

2 participants