[WebProfilerBundle] Make admin button consistent with others #15819
Conversation
stloyd
commented
Sep 16, 2015
stloyd
commented
| Q | A |
|---|---|
| Bug fix? | yes(?) |
| New feature? | no |
| License | MIT |
|
👎 from me. This feature is displayed as a link on purpose. It's a common practice to do that for this kind of secondary utilities in backend designs. Buttons are reserved for important things, and in my opinion this feature is not. |
|
Wait, purging is a GET request ? This should be changed to a POST request IMO |
|
I would even remove the feature from the web interface altogether. I've never used it myself and I'm pretty nobody uses it. |
|
I like the idea of removing this feature. Do other Symfony deciders agree on this? |
|
👍 for removing the feature. If you want to purge, you can do that by emptying whatever your storage is (and for most people, just clearing the cache). |
|
I agree about the removal. I never purge it through the UI. |
|
👍 for removing the feature |
…iler sidebar (javiereguiluz) This PR was merged into the 2.8 branch. Discussion ---------- Removed the "Delete profiles" action from the web profiler sidebar | Q | A | ------------- | --- | Bug fix? | no | New feature? | no | BC breaks? | no | Deprecations? | yes | Tests pass? | yes | Fixed tickets | #15819 | License | MIT | Doc PR | - Commits ------- 8c47eb9 Removed the "Delete profiles" action from the web profiler sidebar
|
I have actually used this (trying to debug a redirect loop and needing the first request) :) but I'm OK with removing this 👍 . |
