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

Add FileReader usage example #37565

Merged
merged 3 commits into from
Jan 28, 2025
Merged

Conversation

alexander-john
Copy link
Contributor

@alexander-john alexander-john commented Jan 9, 2025

…examples (#36443)

Description

Added a detailed description of the FileReader interface, explaining its purpose, usage, and security considerations. Additionally, provided links to related APIs and documentation for further reference.

Motivation

I'm making these changes to address an issue a user was having with the content quality and I hope it provides readers with more resources to better understand this interface.

Related issues and pull requests

Fix #36443

@alexander-john alexander-john requested a review from a team as a code owner January 9, 2025 00:56
@alexander-john alexander-john requested review from sideshowbarker and removed request for a team January 9, 2025 00:56
@github-actions github-actions bot added Content:WebAPI Web API docs size/m [PR only] 51-500 LoC changed labels Jan 9, 2025
@sideshowbarker sideshowbarker removed their request for review January 9, 2025 14:40
Copy link
Contributor

github-actions bot commented Jan 9, 2025

Preview URLs

(comment last updated: 2025-01-28 20:34:52)

@Josh-Cena Josh-Cena changed the title Enhance FileReader documentation with security details and add usage … Add FileReader usage example Jan 28, 2025
Copy link
Member

@Josh-Cena Josh-Cena left a comment

Choose a reason for hiding this comment

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

The example itself looks good to me. Thank you!

@Josh-Cena Josh-Cena merged commit 0e61c49 into mdn:main Jan 28, 2025
8 checks passed
@alexander-john
Copy link
Contributor Author

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content:WebAPI Web API docs size/m [PR only] 51-500 LoC changed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

No examples provided on using the FileReader API
2 participants