Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Email checking widget to DevServices/DevUI #28700

Open
GregJohnStewart opened this issue Oct 20, 2022 · 11 comments
Open

Add Email checking widget to DevServices/DevUI #28700

GregJohnStewart opened this issue Oct 20, 2022 · 11 comments
Labels

Comments

@GregJohnStewart
Copy link
Contributor

GregJohnStewart commented Oct 20, 2022

Description

It would be terribly useful to have a widget on DevServices/ dev ui that one could use to see emails that have been sent by the service.

I was thinking something inspired by the MockMailbox utility used for testing; just simple read-only (from dev ui) list of addresses sent to, and the emails that were sent.

This would be way easier than making devs setup their dev instance with an actual email server.

Implementation ideas

Implementation similar to other dev service widgets.

Maybe a flag, similar to quarkus.mailer.mock that would help control this? Or perhaps if that is on, enable this widget?

@GregJohnStewart GregJohnStewart added the kind/enhancement New feature or request label Oct 20, 2022
@quarkus-bot
Copy link

quarkus-bot bot commented Oct 20, 2022

/cc @geoand, @stuartwdouglas

@geoand
Copy link
Contributor

geoand commented Oct 20, 2022

cc @cescoffier

@cescoffier
Copy link
Member

Good idea!
I wonder if I should wait for @phillip-kruger revamped dev ui first.

@GregJohnStewart
Copy link
Contributor Author

Circling back, what's the state of things? Is the stage set?

Another ticket was added asking for similar capabilities: #32894

@GregJohnStewart GregJohnStewart changed the title Add Email checking widget to DevServices Add Email checking widget to DevServices/DevUI Jun 26, 2023
@geoand
Copy link
Contributor

geoand commented Jun 26, 2023

The new DevUI is in, so it's a matter of priorities now

@phillip-kruger
Copy link
Member

@GregJohnStewart Here are the documentation for the new Dev UI if you are keen to do a PR: https://quarkus.io/version/main/guides/dev-ui-v2

@GregJohnStewart
Copy link
Contributor Author

I can take a stab at it sure. First time for everything :)

@phillip-kruger
Copy link
Member

@GregJohnStewart ping me on Zulip if you have questions.

@melloware
Copy link
Contributor

@GregJohnStewart check out my Extension which does exactly that as a DevService with a UI: https://github.com/quarkiverse/quarkus-mailpit

@cescoffier I think this might be closeable as I am pretty sure this is what was intended.

@cescoffier
Copy link
Member

@melloware did you integrate the mailpit UI (or at least a mail UI) in the dev UI? We want to unified experience.

@codespearhead
Copy link

@cescoffier yes, it's baked into the Dev UI as per this comment.

I wonder if this should be marked as solved now that this PR was merged, because the dependency is from io.quarkiverse.mailpit instead of io.quarkus.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

6 participants