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

Get your Time Table by uploading images #151

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

Conversation

Akshat-Shu
Copy link

@Akshat-Shu Akshat-Shu commented Jan 14, 2025

  1. Image-to-ICS Parsing:

    • Extracts data from uploaded images and parses it directly into .ics format, bypassing the need for HTML as an intermediary.
  2. Image Upload Capability on RollForm Page:

    • Added a new "Upload Image" section to the RollForm page, enabling users to upload images for processing.
  3. Drag-and-Drop Support:

    • Users can now drag and drop images directly onto the upload section, in addition to selecting files via the file picker.

Changelog

  • Added functionality for parsing image data to .ics.
  • Updated the RollForm UI to include an "Upload Image" section.
  • Implemented drag-and-drop support for image uploads.
  • Enhanced file-handling logic to support both drag-and-drop and traditional uploads.

Testing

  • Verified the ICS parsing logic with multiple test images.
  • Ensured backward compatibility with existing features on the RollForm page.

Notes

  • Users must ensure images have clear and structured content for optimal parsing.
  • Further enhancements for error handling and feedback during upload and parsing are planned for future iterations.

Let me know if additional details or changes are needed!

Copy link
Member

@shikharish shikharish left a comment

Choose a reason for hiding this comment

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

Looks good mostly.

app.py Outdated Show resolved Hide resolved
@Akshat-Shu Akshat-Shu requested a review from shikharish January 17, 2025 16:42
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.

2 participants