Skip to content

DRAFT Feature/pdf viewer from main - #84

Draft
suung wants to merge 2 commits into
mainfrom
feature/pdf-viewer-from-main
Draft

DRAFT Feature/pdf viewer from main#84
suung wants to merge 2 commits into
mainfrom
feature/pdf-viewer-from-main

Conversation

@suung

@suung suung commented Jul 29, 2026

Copy link
Copy Markdown
Collaborator
  • Make a branch from main
  • Port pdf viewer component
Screenshot 2026-07-29 at 11 42 56

suung added 2 commits July 28, 2026 13:53
Port from feature/pdf-viewer-with-chunks onto main for #68; omit schema stack.
- Import render_view_report_page in streamlit_app
- Harden AppTest session_state access
- Ruff noqa on touched PDF port files so QG passes
@cursor

cursor Bot commented Jul 29, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@suung
suung marked this pull request as draft July 29, 2026 09:36
dev_server_port = port
dev_server_available = True
break
except:
dev_server_port = port
dev_server_available = True
break
except:
if isinstance(result, str):
try:
result = json.loads(result)
except (json.JSONDecodeError, TypeError):
from report_analyst_enterprise.components.streamlit_component.backend import pdf_viewer

pdf_viewer_available = True
except ImportError:
try:
from report_analyst_enterprise.components.streamlit_component.backend import pdf_viewer
pdf_viewer_available = True
except ImportError:
# Import and call pdf_viewer
from report_analyst_enterprise.components.streamlit_component.backend.pdf_viewer import pdf_viewer

result = pdf_viewer(
left_col, right_col = st.columns([1, 1])
else:
left_col = st.container()
right_col = None
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.

1 participant