Update Configuration settings page images for the Mautic 5.2 UI - #911
Update Configuration settings page images for the Mautic 5.2 UI#911promptless-for-oss wants to merge 2 commits into
Conversation
Refresh 43 of the Configuration settings screenshots with freshly captured Mautic 5.2.0 light-mode screenshots. Sandbox paths/URLs visible in the asset directory, general settings paths, and the tracking script were scrubbed to the generic documentation values. social-settings.png is left unchanged: Mautic 5.2 has no Social settings section in Configuration to capture. Refs mautic#391
RCheesley
left a comment
There was a problem hiding this comment.
@promptless-for-oss some changes needed here.
There was a problem hiding this comment.
@promptless-for-oss this one requires some attention - it seems the PHP IMAP extension isn't enabled on your build environment?
There was a problem hiding this comment.
@promptless-for-oss it looks like the settings may be missing here.
There was a problem hiding this comment.
@promptless-for-oss it looks like the settings may be missing here.
There was a problem hiding this comment.
@promptless-for-oss it looks like the settings may be missing here.
There was a problem hiding this comment.
@promptless-for-oss it looks like the settings may be missing here.
There was a problem hiding this comment.
@promptless-for-oss it looks like the settings may be missing here.
There was a problem hiding this comment.
@promptless-for-oss it looks like the settings may be missing here.
There was a problem hiding this comment.
@promptless-for-oss it looks like the settings may be missing here.
There was a problem hiding this comment.
@promptless-for-oss can we please fix the typo in the filename? SAML not SMAL.
There was a problem hiding this comment.
@promptless-for-oss it looks like the settings may be missing here.
|
Thanks @RCheesley — working through each comment. I'll fix the SMAL → SAML filename typo, and re-capture the screenshots where settings sections came out empty (including checking the PHP IMAP extension in the capture environment). I'll follow up here with the results and a clear note on anything that can't be reproduced in our capture environment. |
|
Mapped all the flagged images to their causes. Most of the "settings missing" shots are the message-queue and email DSN builders captured with the fields left empty, so they show blank example values — re-capturing those with example DSNs filled in to match the previously published images. Your hypothesis on the monitored-inbox screenshot is correct: the PHP IMAP extension wasn't loaded, so Mautic hid the form and showed the 'IMAP extension is required' banner — I'm working on getting IMAP available in the capture environment so that section renders. I'll also fix the SMAL → SAML filename typo. Will follow up with the results. |
Address PR review feedback on the Mautic 5.2 Configuration screenshots: - Recapture the 7 Message queue transport images (sync, Doctrine, Redis, AMQP, Beanstalkd, Amazon SQS, and the failure queue) with their example DSN values populated, so the settings are visible rather than empty. - Recapture the SMTP and SendGrid Email DSN images with example values. - Rename SMAL-settings.png to SAML-settings.png and update its directive. - Restore the previously published monitored-inbox-settings screenshot, since the monitored-inbox form cannot render without the PHP IMAP extension. Refs mautic#391
|
I noticed that some CI checks failed for this PR. I'm investigating whether the failures are caused by this suggestion. If they're unrelated or pre-existing, I'll leave this suggestion unchanged and create a separate suggestion if a standalone docs fix is needed. |
|
Thanks @RCheesley — all your comments are addressed (commit
Separately, heads-up: the failing build check isn't caused by these image changes — it's a pre-existing broken external link ( |
Open in Promptless
Refreshes 43 of the Configuration settings screenshots with freshly captured Mautic 5.2.0 light-mode screenshots — System, Email, API, Asset, Campaign, Contact/Segment/Company, Message queue, Notification, Landing Page, Tracking, Report, Text message, SAML/SSO, and Webhook sections. Sandbox paths/URLs that appeared in default fields (asset-directory path, general-settings Site URL / cache / log paths, and the tracking-script mtc.js URL) were scrubbed to the generic documentation values used in the prior published images.
Scope notes:
social-settings.pngis left unchanged — Mautic 5.2's Configuration has no "Social settings" section (the tab does not exist in this version).sendgrid-api-dsn.pngmirrors the generic Email DSN section because 5.2 has no SendGrid-specific DSN UI.Review feedback addressed (@RCheesley)
Recaptured against a local Mautic 5.2.0 instance with the example DSN values populated, so the settings are visible instead of appearing empty:
queue-disabled.png(sync://),queue-doctrine.png(doctrine://default),queue-redis.png(redis://localhost),queue-amqp.png(amqp://guest:SECRET@localhost:5672),queue-beanstalkd.png(beanstalkd://localhost), andqueue-amazon-sqs.png(sqs://localhost:9494/messages?sslmode=disable). The earlier captures were taken on a fresh install before any DSN was entered, so every field showed empty and the saved DSN readsync://.queue-failures.png— recaptured showing the failure queue configured asdoctrine://default?queue_name=failed.smtp-dsn.pngandsendgrid-api-dsn.png— recaptured with the documented example DSNs filled in (smtp://user:pass@smtp.example.com:portandsendgrid+api://KEY@default).SMAL-settings.png→SAML-settings.pngand updated the.. image::directive indocs/configuration/settings.rst.monitored-inbox-settings.png(PHP IMAP) — you were right: the previous capture only showed the "The PHP IMAP extension is required" banner because IMAP wasn't loaded in the capture environment. The screenshot toolchain runs on PHP 8.4, which removed IMAP from core, and there is no installablephp-imappackage for it, so the Monitored Inbox form cannot be rendered there. I've restored the previously published Monitored Inbox screenshot (which shows the full form) rather than ship the banner-only image. A fresh capture of this one section will need an environment with the IMAP extension available.Closes #391
Trigger Events