Skip to content

feat: cv-file-uploader update to current File object coming from event to internalFile #1609

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

Conversation

isaquebenatti
Copy link
Contributor

@isaquebenatti isaquebenatti commented Jun 20, 2024

Contributes to #1607

What did you do?

Updated to current File object coming from event to internalFile so that if a file has been modified the internalFiles are updated

Why did you do it?

I have case where the user may add a .docx file with the same name but modified, and the current component is not updating with the modified file.

How have you tested it?

With local test environment

Were docs updated if needed?

  • N/A

@isaquebenatti
Copy link
Contributor Author

Hi @davidnixon, could you please review this PR? Thank you! :)

@github-actions github-actions bot added the stale 🍞 No recent activity label Jul 7, 2024
@davidnixon davidnixon removed the stale 🍞 No recent activity label Jul 9, 2024
@davidnixon
Copy link
Contributor

@felipebritor this looks OK to me. Can you have a look?

Copy link
Contributor

@davidnixon davidnixon left a comment

Choose a reason for hiding this comment

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

Just checking with @felipebritor because he knows this component really well.

@felipebritor
Copy link
Contributor

Hello, @davidnixon . I'll test it tonight

@felipebritor
Copy link
Contributor

Looks good, @davidnixon . Sorry for the delay.

@davidnixon davidnixon merged commit 3068ec0 into carbon-design-system:main Jul 17, 2024
5 checks passed
@isaquebenatti isaquebenatti deleted the task/1607-CvFileUploader-improvement branch July 22, 2024 11:55
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.

3 participants