checkpoint: document note preview fix
This commit is contained in:
parent
577e9f4711
commit
9cf4003c31
1 changed files with 10 additions and 3 deletions
|
|
@ -2,7 +2,7 @@
|
||||||
|
|
||||||
## Where things are
|
## Where things are
|
||||||
- Project: `/home/avi/Projects/Keynctr`
|
- Project: `/home/avi/Projects/Keynctr`
|
||||||
- Git repo: `master` @ `cd5d2d7` ("Show published note content inline on Home screen").
|
- Git repo: `master` @ `577e9f4` ("fix: return StoredPublishReport with content to frontend after publish").
|
||||||
- Working tree: clean except the usual untracked items (`.directory`, `.opencode/`,
|
- Working tree: clean except the usual untracked items (`.directory`, `.opencode/`,
|
||||||
`.impeccable/`, `COSMIC_THEME.md`, `KeynectrAppIconPossibility02.jpeg`).
|
`.impeccable/`, `COSMIC_THEME.md`, `KeynectrAppIconPossibility02.jpeg`).
|
||||||
|
|
||||||
|
|
@ -98,8 +98,14 @@
|
||||||
`PublishReport`. Removed `onNavigate` prop from `ComposeScreen` and the
|
`PublishReport`. Removed `onNavigate` prop from `ComposeScreen` and the
|
||||||
"View in Feed" button from both screens. Added `.publish-preview` CSS for the
|
"View in Feed" button from both screens. Added `.publish-preview` CSS for the
|
||||||
content display.
|
content display.
|
||||||
|
19. **Fix: note preview now appears after publishing (this session).** The
|
||||||
|
"Most recent publication" card was not showing the note content preview
|
||||||
|
because the IPC handler returned the bare `PublishReport` (no `content`
|
||||||
|
field) instead of the `StoredPublishReport` that includes it. Changed
|
||||||
|
`src/ipc.rs` to return `stored` instead of `report`.
|
||||||
|
|
||||||
## Commits added in this session (newest first)
|
## Commits added in this session (newest first)
|
||||||
|
- `577e9f4` fix: return StoredPublishReport with content to frontend after publish
|
||||||
- `cd5d2d7` Show published note content inline on Home screen
|
- `cd5d2d7` Show published note content inline on Home screen
|
||||||
- `bab82f5` Persist last publish report across restarts
|
- `bab82f5` Persist last publish report across restarts
|
||||||
- `b0d9143` Add 'View in Feed' button after publishing a note
|
- `b0d9143` Add 'View in Feed' button after publishing a note
|
||||||
|
|
@ -125,8 +131,9 @@ All green in this session, run after the changes:
|
||||||
- Rust: `cargo fmt --check` clean; `cargo test` — 116 passed;
|
- Rust: `cargo fmt --check` clean; `cargo test` — 116 passed;
|
||||||
`cargo clippy --all-targets` — clean, zero warnings; `cargo build --release` — success.
|
`cargo clippy --all-targets` — clean, zero warnings; `cargo build --release` — success.
|
||||||
- Frontend: `npm test` — 15 files / 99 passed; `npm run typecheck` clean; `npm run lint`
|
- Frontend: `npm test` — 15 files / 99 passed; `npm run typecheck` clean; `npm run lint`
|
||||||
clean (only the harmless ES-module reparsing warning); `npm run build` — success
|
clean (only the harmless ES-module reparsing warning); `npm run format:check` has
|
||||||
(Vite bundle built).
|
pre-existing prettier warnings in `HomeScreen.tsx` and `AppProvider.tsx` (not from this
|
||||||
|
session); `npm run build` — success (Vite bundle built).
|
||||||
- Packaging (previous session, still valid artifacts):
|
- Packaging (previous session, still valid artifacts):
|
||||||
`npx electron-builder --linux AppImage deb` produced
|
`npx electron-builder --linux AppImage deb` produced
|
||||||
`frontend/release/Keynctr-0.1.0.AppImage` and
|
`frontend/release/Keynctr-0.1.0.AppImage` and
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue