Skip to content

feat: Nicer /_flags page - #3371

Open
joshlarson wants to merge 5 commits into
mainfrom
jdl/feat/nicer-flags-page
Open

feat: Nicer /_flags page#3371
joshlarson wants to merge 5 commits into
mainfrom
jdl/feat/nicer-flags-page

Conversation

@joshlarson

Copy link
Copy Markdown
Contributor

Scope

No ticket, but I noticed that the helpful banner that tells you what environment you're in was missing from the /_flags page. I figured... while we're here, why not make it look pretty?

Blocked by:

Screenshots

Screenshot 2026-07-29 at 3 28 28 PM

How to test

Go to the flags page, and toggle the flags as you see fit!

@joshlarson
joshlarson requested a review from a team as a code owner July 29, 2026 19:58
@joshlarson
joshlarson requested a review from thecristen July 29, 2026 19:58
@joshlarson joshlarson added do not merge ⏳ it's waiting on something dev-blue Deploy to dev-blue labels Jul 29, 2026
Comment on lines +4 to +7
use Phoenix.VerifiedRoutes,
endpoint: DotcomWeb.Endpoint,
router: DotcomWeb.Router,
statics: DotcomWeb.static_paths()

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.

FYI if you wanted to not need to include this, you can add this to the form

<input type="hidden" name="redirect" value={~p"/_flags"} />

then in the controller can read the redirect param and do

|> redirect(to: params["redirect"])

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Whoa - cool!

@joshlarson
joshlarson enabled auto-merge (squash) July 30, 2026 15:11
@joshlarson joshlarson removed the dev-blue Deploy to dev-blue label Jul 30, 2026
@joshlarson
joshlarson requested a review from thecristen July 31, 2026 14:56
@joshlarson joshlarson removed the do not merge ⏳ it's waiting on something label Aug 1, 2026
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