Skip to content

Emails: Add button to send emails synchronously. - #88

Open
isabellalam12 wants to merge 1 commit into
mainfrom
08-24-emails-add-sync-button
Open

Emails: Add button to send emails synchronously.#88
isabellalam12 wants to merge 1 commit into
mainfrom
08-24-emails-add-sync-button

Conversation

@isabellalam12

Copy link
Copy Markdown
Member

No description provided.

This was referenced Aug 26, 2026

isabellalam12 commented Aug 26, 2026

Copy link
Copy Markdown
Member Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@isabellalam12
isabellalam12 force-pushed the 08-24-emails-add-sync-button branch 2 times, most recently from 5aea21b to a6da2c8 Compare August 26, 2026 17:55
@isabellalam12
isabellalam12 force-pushed the 08-24-emails-remove-defaults branch from 67fb00b to 89ca9db Compare August 26, 2026 17:55
@isabellalam12
isabellalam12 force-pushed the 08-24-emails-add-sync-button branch from a6da2c8 to 76bf469 Compare August 26, 2026 19:11
@isabellalam12
isabellalam12 force-pushed the 08-24-emails-remove-defaults branch 3 times, most recently from 9d0d65f to 8f76576 Compare August 26, 2026 19:27
@isabellalam12
isabellalam12 force-pushed the 08-24-emails-add-sync-button branch from 76bf469 to cd00456 Compare August 26, 2026 19:27
Comment thread js/src/features/emails/_components/SyncEmailSender.tsx
@isabellalam12
isabellalam12 force-pushed the 08-24-emails-remove-defaults branch from 8f76576 to d5e8517 Compare August 26, 2026 19:57
@isabellalam12
isabellalam12 force-pushed the 08-24-emails-add-sync-button branch from cd00456 to 09810e8 Compare August 26, 2026 19:57
@isabellalam12
isabellalam12 force-pushed the 08-24-emails-remove-defaults branch from d5e8517 to 6a582b0 Compare August 26, 2026 20:01
@isabellalam12
isabellalam12 force-pushed the 08-24-emails-add-sync-button branch 2 times, most recently from 606a1d9 to ba9446c Compare August 26, 2026 20:12
@isabellalam12
isabellalam12 force-pushed the 08-24-emails-remove-defaults branch from 6a582b0 to b10d002 Compare August 26, 2026 20:16
@isabellalam12
isabellalam12 force-pushed the 08-24-emails-add-sync-button branch 2 times, most recently from 7496a96 to 8b5194d Compare August 26, 2026 20:23
@isabellalam12
isabellalam12 force-pushed the 08-24-emails-remove-defaults branch 3 times, most recently from cc49247 to 19caa29 Compare August 26, 2026 20:29
@isabellalam12
isabellalam12 force-pushed the 08-24-emails-add-sync-button branch from 8b5194d to bd5c4e3 Compare August 26, 2026 20:29
@isabellalam12 isabellalam12 changed the title Add button to send emails synchronously. Emails: Add button to send emails synchronously. Aug 26, 2026
@isabellalam12
isabellalam12 changed the base branch from 08-24-emails-remove-defaults to graphite-base/88 September 3, 2026 19:39
@isabellalam12
isabellalam12 force-pushed the 08-24-emails-add-sync-button branch from bd5c4e3 to 44ce8d9 Compare September 3, 2026 19:39
@isabellalam12
isabellalam12 changed the base branch from graphite-base/88 to main September 3, 2026 19:39
@isabellalam12
isabellalam12 force-pushed the 08-24-emails-add-sync-button branch 4 times, most recently from d60b7ea to 664f9a7 Compare September 3, 2026 20:04
@isabellalam12
isabellalam12 marked this pull request as ready for review September 3, 2026 20:07
@isabellalam12
isabellalam12 requested a review from a team September 3, 2026 20:07
@isabellalam12
isabellalam12 requested review from spiffyy99 and removed request for Arshadul-Monir September 3, 2026 20:07
@graphite-app

graphite-app Bot commented Sep 3, 2026

Copy link
Copy Markdown

Graphite Automations

"Request reviewers once CI passes" took an action on this PR • (09/03/26)

2 reviewers were added to this PR based on Henry Chen's automation.

Comment thread js/src/features/emails/_components/EmailSender.tsx Outdated
Comment thread src/main/java/org/patinanetwork/patchats/email/EmailService.java Outdated
Comment thread src/main/java/org/patinanetwork/patchats/email/EmailService.java Outdated
@isabellalam12
isabellalam12 force-pushed the 08-24-emails-add-sync-button branch from 664f9a7 to bce8637 Compare September 3, 2026 20:40
Comment thread js/src/features/emails/_components/EmailSender.tsx Outdated
Comment thread js/src/features/emails/_components/SyncEmailSender.tsx Outdated
Comment thread src/main/java/org/patinanetwork/patchats/email/EmailService.java
@spiffyy99

Copy link
Copy Markdown

Have a screenshot of what this looks like from the front end?

Copy link
Copy Markdown
Member Author

Have a screenshot of what this looks like from the front end?

Screenshot 2026-09-04 at 1.12.55 PM.png

@isabellalam12
isabellalam12 force-pushed the 08-24-emails-add-sync-button branch 2 times, most recently from 9f9b936 to f36b39a Compare September 7, 2026 16:57

Copy link
Copy Markdown
Member Author

Screenshot 2026-09-07 at 1.00.13 PM.pngScreenshot 2026-09-07 at 12.56.21 PM.png

Updated UI with left padding, more descriptive buttons, and hover text.

@isabellalam12
isabellalam12 force-pushed the 08-24-emails-add-sync-button branch from f36b39a to 36fbb34 Compare September 7, 2026 17:01
Comment thread src/main/java/org/patinanetwork/patchats/email/EmailService.java

isabellalam12 commented Sep 7, 2026

Copy link
Copy Markdown
Member Author

Merge activity

  • Sep 7, 5:07 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Sep 7, 5:08 PM UTC: Graphite rebased this pull request as part of a merge.
  • Sep 7, 5:09 PM UTC: Graphite couldn't merge this PR because it failed for an unknown reason.
  • Sep 7, 5:11 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Sep 7, 5:11 PM UTC: Graphite couldn't merge this PR because it failed for an unknown reason.
  • Sep 7, 5:33 PM UTC: Graphite couldn't merge this PR because it failed for an unknown reason.
  • Sep 7, 5:35 PM UTC: Graphite couldn't merge this PR because it failed for an unknown reason.

@isabellalam12
isabellalam12 force-pushed the 08-24-emails-add-sync-button branch from 36fbb34 to 7ff9d0d Compare September 7, 2026 17:07
@sonarqubecloud

sonarqubecloud Bot commented Sep 7, 2026

Copy link
Copy Markdown

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