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

feat: GitHub Link Preview Details (Optimised & light) #487

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

Conversation

ARYPROGRAMMER
Copy link

This Adds the Feature Mentioned in Issue#328
This PR also Solves the Issue of App Being Slow as mentioned in the Previous PR Here

Specifically, for all stories with a GitHub Repo preview I have added an Enhancement to the existing UI to fetch the details of the repo specified and give its preview without even opening it. Thereby providing the user a more seamless experience.

  • Added Bloc State Management for Github Repo Fetching to Reduce Core time lag and latency with seamless UX
  • Since no FutureBuilder is used only one API call is given reducing the widget tree overflow errors
  • All features are Working
  • Free and Flexible to Change further, Main Logic Implemented Separately
  • No Breaking Changes
  • Added Enhancement requires Admin's GitHub PAT (Personal Access Token)
  • No Render Issues or Time Lags Observed

Sample Image :

Realtime App View

Sample Video :

Realtime Video

Co-authored-by: Jojo Feng <georgefung78@live.com>
@ARYPROGRAMMER
Copy link
Author

ARYPROGRAMMER commented Oct 24, 2024

@Livinglist I request you to kindly perform the minor changes before releasing, and also that the feature is decoupled from repo , you can merge this and then flexibly change it around as your convention and needs as I tend to fall short and will most probably break some underlying architecture. Moreover the function needs only a little refinement which only you could do.

@ARYPROGRAMMER
Copy link
Author

@Livinglist can you run ci for this and get this going

@ARYPROGRAMMER
Copy link
Author

@Livinglist any updates on this?

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