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

Set gzip default based on code reloading in endpoint #5971

Conversation

rmoorman
Copy link
Contributor

@rmoorman rmoorman commented Nov 9, 2024

Adjust the default for the gzip flag in the generated endpoint to take
code reloading into account.

Fixes #5960

Introduce a `:success` kind for flash messages in the generator
output and update relevant documentation.

Success messages that previously used `:info` now use `:success`,
ensuring they continue to appear in green with a "Success!" title.

In certain cases (e.g. password reset), `:info` is retained to avoid
disclosing too much information (such as the existence of an account).
These messages now appear in blue with a "Please note" title.

Rationale for those changes:

- Differentiate informational messages from successful actions

- Avoid confusion from informational messages styled in green, which
  can imply success

- Ensure `:info` is used for neutral messages, and use `:success` to
  clearly indicate completed actions
Adjust the default for the gzip flag in the generated endpoint to take
code reloading into account.

Fixes phoenixframework#5960
@rmoorman
Copy link
Contributor Author

rmoorman commented Nov 9, 2024

Oops. Based off of a wrong branch :-) my bad

@rmoorman rmoorman closed this Nov 9, 2024
@rmoorman rmoorman deleted the endpoint-plug-static-gzip-and-code-reloading branch November 9, 2024 18:00
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.

Improve default for Plug.Static usage in endpoint
1 participant