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

refactor: refactor use progress hook #241

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Commits on May 23, 2024

  1. refactor: refactor use progress hook

    make handle as const there is no need to state
    add interface progresshook
    verification xhr.lengthComputable valide to calclul new progress
    create handleUploadProgress in use effect to create it only when we needed and Encapsulated it
    verify new progress in the handleuploadprogress to lake it clean and to avoid extra effect
    Cleanup event listener on unmount or when files change
    Iyadchafroud committed May 23, 2024
    Configuration menu
    Copy the full SHA
    dd46b7f View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    0781657 View commit details
    Browse the repository at this point in the history

Commits on May 26, 2024

  1. refactor: refactoring checkFileType function

    add JSDoc .
    return `true` immediately if any file or criter to match  for improving performance.
    accepted file types to lowercase to be sure about case-insensitive
    Removed unnecessary regex and replaced it with simple method
    Encapsulated  logic in Array.some()
    Iyadchafroud committed May 26, 2024
    Configuration menu
    Copy the full SHA
    3ad03cf View commit details
    Browse the repository at this point in the history

Commits on May 27, 2024

  1. Configuration menu
    Copy the full SHA
    dd105c9 View commit details
    Browse the repository at this point in the history