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

[InputFile] Set last modified date #3044

Merged
merged 3 commits into from
Dec 14, 2024
Merged

[InputFile] Set last modified date #3044

merged 3 commits into from
Dec 14, 2024

Conversation

maNobly
Copy link
Contributor

@maNobly maNobly commented Dec 12, 2024

Pull Request

πŸ“– Description

Making the property LastModified from the IBrowserFile available on the the FluentInputEventArgs object, so it can be accessed.

🎫 Issues

πŸ‘©β€πŸ’» Reviewer Notes

πŸ“‘ Test Plan

βœ… Checklist

General

  • I have added tests for my changes.
  • I have tested my changes.
  • I have updated the project documentation to reflect my changes.
  • I have read the CONTRIBUTING documentation and followed the standards for this project.

⏭ Next Steps

Save the property LastModified from the IBrowserFile to the FluentInputEventArgs object.
@maNobly
Copy link
Contributor Author

maNobly commented Dec 12, 2024

@microsoft-github-policy-service agree

@dvoituron
Copy link
Collaborator

Have you verified that this `File.LastModified' property is also available in a WASM project?

@maNobly
Copy link
Contributor Author

maNobly commented Dec 14, 2024

Have you verified that this `File.LastModified' property is also available in a WASM project?

Yes, I have also tested in WASM project and the prop is available.

@dvoituron dvoituron enabled auto-merge (squash) December 14, 2024 17:55
@dvoituron dvoituron merged commit 7fa9f4a into microsoft:dev Dec 14, 2024
4 checks passed
@vnbaaij vnbaaij added this to the v4.11 milestone Dec 19, 2024
@vnbaaij vnbaaij changed the title [FluentInput] Set last modified date [InputFile] Set last modified date Dec 19, 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.

3 participants