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

Fixed a bug that caused different fileReader to be selected #821

Merged
merged 1 commit into from
Apr 2, 2024

Conversation

JIIOryo
Copy link
Contributor

@JIIOryo JIIOryo commented Apr 2, 2024

Summary

Fixed a bug that could cause different FileReaders to be selected when using multiple FileReaders.

The cause of this bug is that the same id could be assigned to adjacent FileReaders due to the use of Date.now() for id generation.

Before

before

After

after

Related PullRequest

#755

issue

#779

Copy link

changeset-bot bot commented Apr 2, 2024

🦋 Changeset detected

Latest commit: 125e009

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@viron/app Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@nonoakij nonoakij self-requested a review April 2, 2024 07:57
@nonoakij nonoakij linked an issue Apr 2, 2024 that may be closed by this pull request
Copy link
Contributor

@nonoakij nonoakij left a comment

Choose a reason for hiding this comment

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

Thank you @JIIOryo .
I can't say for sure that Math.random() will result in a different ID, but it is better than the current Date.now.

@nonoakij nonoakij merged commit 92dd3b4 into cam-inc:develop Apr 2, 2024
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.

uploading two files does not work
2 participants