feat(feedback): client-side journal export and email handoff - #3705
Merged
Merged
Conversation
Replace POST /feedback/upload with POST /feedback/export returning the journal upload envelope. Remove remote forward stub and update unit tests. Co-authored-by: Guillaume De Saint Martin <GuillaumeDSM@users.noreply.github.com>
Call POST /feedback/export, build node_journal.zip in the browser, download it, and open a mailto draft. Update Share feedback CTA and Vitest. Co-authored-by: Guillaume De Saint Martin <GuillaumeDSM@users.noreply.github.com>
Add route registration regression check for feedback export API. Co-authored-by: Guillaume De Saint Martin <GuillaumeDSM@users.noreply.github.com>
cursor
Bot
force-pushed
the
cursor/feedback-sharing-export-e092
branch
from
September 22, 2026 18:18
ba15fff to
4a9d823
Compare
Co-authored-by: Guillaume De Saint Martin <GuillaumeDSM@users.noreply.github.com>
GuillaumeDSM
force-pushed
the
cursor/feedback-sharing-export-e092
branch
from
September 22, 2026 19:44
4a9d823 to
35741f6
Compare
GuillaumeDSM
marked this pull request as ready for review
September 22, 2026 19:44
GuillaumeDSM
force-pushed
the
cursor/feedback-sharing-export-e092
branch
from
September 22, 2026 20:57
35741f6 to
d3e3b49
Compare
Open
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
3 participants
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.
Summary
Implements Notion feedback sharing card: Node no longer uploads feedback to a remote server. The API exposes
POST /api/v1/feedback/export(JSON journal envelope only). The Share feedback UI downloadsnode_journal.zip(singlenode_journal.jsonmember) and opens amailto:draft tocontact@octobot.cloud.Changes
/feedback/uploadwith/feedback/export; remove remote forward stub; keepGET /feedback/previewas JSON.install_id/app_versionin mail body); recovery/degraded still uses client fallback envelope.test_feedback_api.py+ Vitest (feedback-share, dialog).Verification
tests/unit_tests/community/node_journal/test_feedback_api.py(12 tests)npm testinnode_web_interface(830 tests)python -m tools.extended_linter --base origin/dev --skip-tentacles-reinstall(OK; full reinstall blocked: no Python 3.13 / cloud-install L2 on this VM)build_openapi.py+npm run generate-client(generated files gitignored)Artifacts
artifacts/phase-2-share-feedback-dialog.pngartifacts/phase-2-share-feedback-mailto-ux.png(Live agent-seed UI on
:8000was blocked:node_api_interfacetentacle did not load in this environment; captures document dialog + mailto flow.)Commits
a3bbda4d— backend export JSON + pytest067aa1ad— frontend zip + mailto + Vitest97a2842b— route registration regression test