- last update
over 1 year ago Composer require failure - 🇮🇳India shobhit_juyal New Delhi
Hi,
I faced the same issue and added a patch for it. Kindly review.
- Status changed to Needs work
30 days ago 9:29am 21 December 2024 - 🇩🇪Germany Anybody Porta Westfalica
@shobhit_juyal could you please provide this as MR?
- Merge request !25Issue #3328079: Redirect users to specific page after login. → (Open) created by sayan_k_dutta
- 🇺🇸United States smustgrave
Should be using DI.
Also @sayan_k_dutta would recommend testing and reviewing patches vs just turning to MRs in order to receive credit.
- 🇮🇳India sayan_k_dutta
Made the requested changes.
Tested the changes. Working as expected. Attaching a screenshot. - 🇺🇸United States smustgrave
Please make sure everything passes before moving to review
- 🇮🇳India sayan_k_dutta
Seems like RestrictIpAcessTest.php is not correctly executing the test for Blacklisted paths. I don't have much knowledge about unit tests. Hence unassigning myself. Anyone please look into it to move it forward.