Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add HTTP method to route:list #1411

Open
peterjaap opened this issue Feb 16, 2024 · 2 comments
Open

Add HTTP method to route:list #1411

peterjaap opened this issue Feb 16, 2024 · 2 comments

Comments

@peterjaap
Copy link
Contributor

Action controllers should implement either HttpGetActionInterface, HttpPostActionInterface, or both (when no interface is implemented it also supports both).

I propose adding a column that shows that info;

$ magerun2 route:list | grep elgentos                       
| frontend  | elgentos_ups                    | Elgentos_Ups                        | elgentos_ups/ajax/upslocations                                    |    POST         |
| frontend  | elgentos_ups                    | Elgentos_Ups                        | elgentos_ups/ajax/updateupslocation                               |    POST, GET    |
| adminhtml | elgentos_mandrill               | Elgentos_Mandrill                   | elgentos_mandrill/account/test                                    |    GET          |
| adminhtml | elgentos_customproduct          | Elgentos_CustomProduct              | elgentos_customproduct/ajax/tax                                   |    POST, GET    |
cmuench added a commit to cmuench/n98-magerun2 that referenced this issue Apr 27, 2024
cmuench added a commit that referenced this issue Apr 27, 2024
@cmuench
Copy link
Member

cmuench commented Apr 27, 2024

@peterjaap I added this feature to the develop version.

@peterjaap
Copy link
Contributor Author

@cmuench love it! On Kingsday 🤴 even!

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

No branches or pull requests

2 participants