Skip to content

Feat: Add health check box on admin page- ref #6738 #6879

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Feat: Add health check box on admin page- ref #6738 #6879

wants to merge 4 commits into from

Conversation

Coussecousse
Copy link
Contributor

@Coussecousse Coussecousse commented Oct 8, 2025 •
edited
Loading

Hi !

I just added a health check box just as you requested (I think) in #6738 😄

Check if :

  • The mail DSN or the origin sender e-mail are not defined
  • If there are more than one active url, the global admin should be in it.

Screens :

  • Nothing is good 😞 :
Capture d’écran du 2025-10-08 14-45-38
  • URL is active and Admin is not register in it (:
Capture d’écran du 2025-10-08 14-21-04
  • Email is ok :
Capture d’écran du 2025-10-08 14-20-51
  • All is good 😄 :
Capture d’écran du 2025-10-08 14-20-22

⚠️ I don't know why, but mailer_from_email seems to be mixed up...
Proofs :
Capture d’écran du 2025-10-08 14-47-52
Capture d’écran du 2025-10-08 14-41-43

I noticed it when I was doing my test. I will investigate

erika added 4 commits October 8, 2025 13:47
Feature: Add health check block to admin index for email
a0b5647
Feat: Add health check for admin URL access
55c9ebf 
Fix: Get active urls instead of all urls and get them from db

If we don't get them from db, changes aren't apply

Fix: Update mail settings check to handle 'null://null' as a valid empty value
Minor: Move health check block after the system block
32b5ff9
Minor: ECS
53a902a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@Coussecousse