Skip to content

Editing a sent report: no way to add a flash dump, and the page doesn't say where one goes #456

Closed

Description

@openipc-ai

From the user group, 2026-10-11 (a user, trying the new report editor on /club):

можно файлом добавить только «Фото / Лог загрузки / Консоль U-Boot», добавить бинарник прошивки нельзя
(only Photo / Boot log / U-Boot console can be added as a file; a firmware binary cannot)

What the code does

  • The editor's add-file kinds are photo, boot_log, uboot_env (frontend/apps/site/src/components/club/EditReport.tsx, ADD_KINDS).
  • The server refuses backup and yaml parts in any edit (service/internal/reports/handler.go, in read, edit branch): "cannot be edited: an edit changes the note, the camera proposed, and adds or removes photos and text files".
  • The rule's stated reason (service/internal/reports/edit.go): ipctool's output and a backup are what the camera said and stay as sent. That is about changing or removing them; removing a backup is refused separately.

The gap
Adding a dump to a report that has none falls under the same refusal, and the editor doesn't say what to do instead. Today the dump can only arrive as a new report: the board panel's "Have this board?" form (Full flash dump), or ipctool with a club code that joins the existing report (ipctool upload --note club-XXXX-XXXX).

To decide

  1. Allow adding a backup (and/or ipctool's YAML) to a report that has neither, sent back to review like any edit and asked for consent (public/private) as the send form does. Or:
  2. Keep the refusal, and say on the editor how to send the dump: the board's send form, or a joining ipctool code.

See also the issue on what a programmer dump earns compared with an ipctool backup.

Activity

  1. openipc-ai commented on Oct 11, 2026

    @openipc-ai
    CollaboratorAuthor

    Live on openipc.org (#461). In the report editor on /club, a report sent without a flash dump now offers Full flash dump under Add:

    • one dump per report; a dump a report already has stays as it was sent
    • a whole chip read with a programmer, or the ipctool backup taken with the report's own ipctool output
    • private unless you tick "publish", as on the send form
    • a report already reviewed goes back for review, and the dump earns its stars then

    ipctool's own output still can't be added by an edit. Send it from the camera with a club code that joins the report (ipctool upload --note club-XXXX-XXXX). What a programmer dump should earn is #458.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions