Repository navigation
feat: show opportunity overview in application review - #257
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Application review opens on Opportunity, showing the company, role, location, work mode, pay, and captured posting summary, responsibilities and qualifications. Tabs are Opportunity, Resume, Cover letter and Q&A. Original and full posting are available from the overview; the full posting reuses the existing modal read-only and carries the resolved company relation.
Packet generation and approval fingerprints remain internal. Material review controls appear on material tabs, and explicit final-submission approval and current-material gates are preserved. No employer submissions were made.
Current source includes main at
c25be231(v0.9.0); the feature patch was identical after integration, then the live-data company-relation fix was added. Independent standard review plus targeted recalls are CLEAN through299851ba232c04ee123720e13cbaa461c18d0265(balanced Claude Sonnet, same review cycle). Coverage:/tmp/iolaus-256-evidence/coverage.json.Validation: current
pnpm format-check,pnpm lint,pnpm check,pnpm build, browser integration tests (3) and strict SMRT freshness pass. Focused review/approval tests (41) and the four application-preparation Playwright scenarios pass on the integrated source across Android portrait, landscape, narrow and desktop. The real-shape separate-company regression fails before its fix and passes after. CI initially caught duplicate headings and outdated visible-Packet E2E expectations; those are fixed while retaining internal packet PDF checks.Local full-suite runs on the latest-main integration hit changing, unrelated five-second SQLite timer failures under workstation load (4,760 tests passed on the bounded run); these are not counted as green. Exact-head workspace/full-suite, mobile-e2e and candidate-image CI must pass before merge/release.
Closes #256