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

fix: format failed to get document text #56

Merged
merged 2 commits into from
Jul 6, 2024

Conversation

aamirazad
Copy link
Owner

@aamirazad aamirazad commented Jun 30, 2024

TL;DR

This fixes the broken formatting of the error when the pdf doesn't load.

What changed?

  • Updated the document-viewer.tsx component to display an error message with styling when a document fails to load

Why?

The previous error message was unstyled and not very user-friendly. This update improves the user experience when a document fails to load.

Copy link

vercel bot commented Jun 30, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
homelab-connector ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 6, 2024 5:20pm

@aamirazad aamirazad marked this pull request as ready for review June 30, 2024 16:27
@aamirazad aamirazad force-pushed the 06-29-fix_clicking_back_doesn_t_re_query_search branch from 0c0f823 to fe2129d Compare June 30, 2024 17:05
@aamirazad aamirazad force-pushed the 06-30-fix_format_failed_to_get_document_text branch from 22f8c97 to 24be218 Compare June 30, 2024 17:05
@aamirazad aamirazad force-pushed the 06-30-fix_format_failed_to_get_document_text branch from 24be218 to 3f27540 Compare July 1, 2024 13:09
@aamirazad aamirazad force-pushed the 06-29-fix_clicking_back_doesn_t_re_query_search branch from 191301f to e5a70b1 Compare July 6, 2024 17:06
@aamirazad aamirazad force-pushed the 06-30-fix_format_failed_to_get_document_text branch from 3f27540 to 2396262 Compare July 6, 2024 17:06
Copy link
Owner Author

aamirazad commented Jul 6, 2024

Merge activity

  • Jul 6, 1:12 PM EDT: @aamirazad started a stack merge that includes this pull request via Graphite.
  • Jul 6, 1:20 PM EDT: Graphite rebased this pull request as part of a merge.
  • Jul 6, 1:21 PM EDT: @aamirazad merged this pull request with Graphite.

@aamirazad aamirazad force-pushed the 06-29-fix_clicking_back_doesn_t_re_query_search branch from e5a70b1 to 344385a Compare July 6, 2024 17:17
Base automatically changed from 06-29-fix_clicking_back_doesn_t_re_query_search to main July 6, 2024 17:19
@aamirazad aamirazad force-pushed the 06-30-fix_format_failed_to_get_document_text branch from 2396262 to aa9bf83 Compare July 6, 2024 17:19
@aamirazad aamirazad merged commit 1d9fd71 into main Jul 6, 2024
5 checks passed
@aamirazad aamirazad deleted the 06-30-fix_format_failed_to_get_document_text branch July 6, 2024 17:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix Something isn't working patch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant