[GitHub] [apisix-dashboard] liuxiran commented on a change in pull request #1701: fix: user login request should remove its own prefix option

[GitBox]

liuxiran commented on a change in pull request #1701:
URL: https://github.com/apache/apisix-dashboard/pull/1701#discussion_r607407406



##########
File path: web/src/pages/User/components/LoginMethodPassword.tsx
##########
@@ -105,7 +105,6 @@ const LoginMethodPassword: UserModule.LoginMethod = {
         const result = await request('/apisix/admin/user/login', {
           method: 'POST',
           requestType: 'json',
-          prefix: '',

Review comment:
       it seems that you have to modify the request url to `/user/login` at the 
same time, otherwise, `/apisix/admin` will appear twice under the global prefix 




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org