Skip to content

CP-14721 - Show success toast after DocuSeal document upload - #82

Merged
bernardodsanderson merged 2 commits into
mainfrom
CP-14721
Jul 31, 2026
Merged

CP-14721 - Show success toast after DocuSeal document upload#82
bernardodsanderson merged 2 commits into
mainfrom
CP-14721

Conversation

@bernardodsanderson

Copy link
Copy Markdown
Collaborator

CP-14721

What

After uploading or replacing a document in a DocuSeal task form, users now see a short success toast confirming the upload worked

Why

Upload could finish with no clear feedback, so people weren’t sure whether the document was saved before they continued editing the form

How to test

  1. Run DocuSeal (foreman start -f Procfile.dev) and open a form at http://localhost:3001
  2. On an empty form, upload a PDF/image via the dropzone — expect “Document uploaded successfully” bottom-right, then auto-dismiss ~4s
  3. Replace an existing document — expect the same toast once
  4. Optional: blank page should not toast; multi-file replace should toast once, not once per file

Screenshot

Screenshot 2026-07-30 at 4 19 56 PM

What
After uploading or replacing a document in a DocuSeal task form, users now see a short success toast confirming the upload worked

Why
Upload could finish with no clear feedback, so people weren’t sure whether the document was saved before they continued editing the form

How to test
1. Run DocuSeal (foreman start -f Procfile.dev) and open a form at http://localhost:3001
2. On an empty form, upload a PDF/image via the dropzone — expect “Document uploaded successfully” bottom-right, then auto-dismiss ~4s
3. Replace an existing document — expect the same toast once
4. Optional: blank page should not toast; multi-file replace should toast once, not once per file
Comment thread db/schema.rb Outdated
t.index ["account_id", "event_datetime"], name: "index_email_events_on_account_id_and_event_datetime"
t.index ["email"], name: "index_email_events_on_email"
t.index ["email"], name: "index_email_events_on_email_event_types", where: "((event_type)::text = ANY ((ARRAY['bounce'::character varying, 'soft_bounce'::character varying, 'complaint'::character varying, 'soft_complaint'::character varying])::text[]))"
t.index ["email"], name: "index_email_events_on_email_event_types", where: "((event_type)::text = ANY (ARRAY[('bounce'::character varying)::text, ('soft_bounce'::character varying)::text, ('complaint'::character varying)::text, ('soft_complaint'::character varying)::text]))"

@ryanarakawa ryanarakawa Jul 30, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Is this just a weird Node/Postgres mismatch or something? I vaguely remember this would happen every now and then at the beginning of the project.

@ryanarakawa ryanarakawa left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks good, that schema diff might be something to discard or look into in the future.

## What
Revert the accidental schema.rb formatting change from the upload toast PR

## Why
The index SQL rewrite came from a local dump/version difference and is unrelated to the toast work

## How to test
Confirm PR no longer includes db/schema.rb; toast behavior unchanged
@bernardodsanderson
bernardodsanderson merged commit 67ef382 into main Jul 31, 2026
5 checks passed
@bernardodsanderson
bernardodsanderson deleted the CP-14721 branch July 31, 2026 17:41
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.

2 participants