Discarded Loading Without Input and Added an Alert for Missing Data

[sakina1303]

This pull request addresses the issue #142 .

Changes Implemented:

Input Validation:

-Added a check to ensure at least one input (text, file, or URL) is provided before proceeding.

Alert Message:

-Displays a concise and professional alert: "Please provide input (text, file, or URL) to generate your quiz."

Prevent Loading:

-Blocked the loading spinner from displaying unless the input validation passes.

Code Refactor:

-Updated the handleSaveToLocalStorage function to include validation logic for input fields.

Benefits:

-Improved User Experience: Guides the user to provide input, reducing confusion.

-Backend Optimization: Prevents empty requests, saving server resources.

-Error Prevention: Ensures meaningful data is sent to the backend.

-Professional Feedback: Aligns application behavior with standard UI/UX practices.

Testing:

• Manual tests conducted to verify:

1. Alert displays when no input is provided.
2. Loading does not proceed without input.
3. Normal functionality works seamlessly when input is valid.

Before changes:

After changes:

Fixes #142

[sakina1303]

Hi Sir's @Roaster05 @Aditya062003 ,

Awaiting your reviews and let me know if further improvement needed .
Thanks !

[Aditya062003]

Hey, pls mimic these changes in the extension too

[sakina1303]

Hey, pls mimic these changes in the extension too

for sure sir! thank you.

[sakina1303]

Hey, pls mimic these changes in the extension too

Hi Sir @Aditya062003 , I've implemented the required changes in the extension as well.

Before :

After:

Please check it out.

Thanks!

[sakina1303]

@Aditya062003 sir I've done the necessary changes in extension as well as you mentioned. please check it out. thanks!

[Aditya062003]

Great! Could you just revert the changes for package-lock.json, seems unnecessary. Otherwise, looks good to go!

[sakina1303]

Great! Could you just revert the changes for package-lock.json, seems unnecessary. Otherwise, looks good to go!

sure sir, i'm updating it.

[sakina1303]

Hey @Aditya062003 sir, I've reverted the changes for package-lock.json, please check it out. Thanks!