Repository navigation
Add manuscript preview to the shared-account landing page - #16
Conversation
DebelToni
left a comment
There was a problem hiding this comment.
Integration review: needs attention before merge.
- This revision is still based on
010beb9; main is nowac6c8d0, and GitHub reports merge conflicts. Please rebase and resolve the AuthPage/styles integration before requesting review of the resulting revision. - Preserve the current site-account flow: the Continue to shared accounts link, safe deep-link return path and expired-attempt handling. The proposed AuthPage/test snapshot still contains the retired local username/password/invitation forms and api.login/api.register assumptions. Update those tests for the central account redirect; do not restore local credentials, invite-only account creation or localStorage human sessions.
- Keep the descriptive Collaborative LaTeX heading and minimal useful copy. The proposed One manuscript / Many minds slogan is outside the agreed UI direction.
The manuscript example can be preserved in a compatible rebased revision. No PR code, tests, build or installer was executed on DGX. This is a blocked integration review, not approval of the complete UI revision; the new head/base will need review and hosted CI.
b85c9e2 to
f03ab14
Compare
DebelToni
left a comment
There was a problem hiding this comment.
Reviewed the complete seven-file diff at f03ab14 against current main ac6c8d0. No blocking findings.
The previous integration findings are addressed: rebased conflict-free; site-account redirect, safe return target and explicit expired-attempt recovery retained; no local password/invitation form or human token storage; descriptive Collaborative LaTeX heading retained. The manuscript/source example is fixed local JSX/text with no network calls, HTML injection, document access or additional permissions. Reviewed view/file controls, focus/pressed semantics, responsive CSS, dark-theme variables and the scoped removal of obsolete auth styles; dashboard styling is preserved.
Hosted exact-head test/analyze and CodeQL checks passed. No candidate code, tests, dependencies, hooks or builds were executed on DGX. No backend, dependency, workflow, security policy or host-automation changes. Visual rendering on the eventual live release remains a deployment check.
Add an interactive manuscript example to the shared-account landing page, with a responsive layout, clearer spacing, and the existing light/dark themes. Visitors can switch between a paper and its modular LaTeX files; the sign-in action stays ahead of the example on phones.
Based on main ac6c8d0. The descriptive Collaborative LaTeX heading and concise copy are unchanged. The current Continue to shared accounts anchor, validated deep-link return path, and static expired-attempt/provider recovery remain unchanged. No local credentials, invitations, or human bearer-token sessions are restored. Auth styling lives alongside the page; dashboard and workspace rules are preserved.
Validation:
No new dependencies, API calls, permissions, migrations, provider configuration, or storage. The example uses only local component state. Hosted CI and CodeQL must pass for this new revision; real Docker compiler checks run there because the local Docker daemon is unavailable. Existing owner-controlled review and deployment gates remain in effect.