Skip to content

Doc2FAQ v0.5: source viewer, review status, and combined filters - #13

Merged
OKAN-SOFTWARE merged 2 commits into
mainfrom
feature/doc2faq-v0.5-review
Nov 6, 2026
Merged

OKAN-SOFTWARE merged 2 commits into
mainfrom
feature/doc2faq-v0.5-review

Conversation

@OKAN-SOFTWARE

@OKAN-SOFTWARE OKAN-SOFTWARE commented Nov 6, 2026 •

Copy link
Copy Markdown
Owner

Changes

Select an FAQ to highlight all of its exact original passages beside the answer editor on desktop, with a stacked source viewer on mobile. Complete FAQs can be marked Approved or Needs review; editing either field or merging resets approval.

Combine question/answer search with status filters and visible/approval counts. Filters affect only the review list: saves, export previews, and every export format retain all FAQs in draft order. Hidden incomplete entries still block export.

Review status survives project downloads and optional autosave. Schema-1 projects upgrade with Needs review defaults; new schema-2 saves require v0.5 or later. Development builds use 0.5.0-SNAPSHOT; v0.4.0 remains the published download.

Review fix

Typing in the question or answer now preserves the source viewer DOM and scroll position. Only explicitly opening or changing the source selection scrolls to a highlighted passage. A regression test reproduced the original issue; the fix passed a real mobile browser check with multiple source passages and the full desktop/mobile smoke checks.

Validation

  • GitHub Actions Java 21 build passed: 50 Java tests, including PDF/Word export integration checks, with zero failures/errors/skips.
  • JavaScript suite passed locally and in CI: 45 tests, including approval restore/reset, legacy-project migration, combined filters, exact source highlights, incomplete-entry handling, and existing request/export checks.
  • Chromium browser smoke checks passed at 1440px and 390px: source viewer positioning, no horizontal overflow, safe text rendering, combined search/status filters, autosave/reload, and approval reset.
  • Browser JSON download retained the full draft while filtered. Mocked PDF/DOCX request payloads also retained both FAQs; actual binary export tests passed in the Java integration suite.
  • git diff --check passed.

CI: https://github.com/OKAN-SOFTWARE/Doc2FAQ/actions/runs/42272623286

The release checklist covers the new workflow. Packaged-JAR manual smoke testing and release publication remain separate; no release has been published.

@OKAN-SOFTWARE
OKAN-SOFTWARE marked this pull request as ready for review November 6, 2026 18:35
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Nov 6, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-11-06T18:36:47.371432Z 1e7037c Draft marked ready
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 1e7037c484

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread src/main/resources/static/app.js Resolved
@OKAN-SOFTWARE
OKAN-SOFTWARE merged commit f700923 into main Nov 6, 2026
2 checks passed
@OKAN-SOFTWARE
OKAN-SOFTWARE deleted the feature/doc2faq-v0.5-review branch November 6, 2026 18:47
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