kyoty commented on pull request #5700:
URL: https://github.com/apache/dolphinscheduler/pull/5700#issuecomment-872367658


   @chengshiwen  @CalvinKirs @myangle1120 Sorry to bother you guys.
   + this PR is supposed to solve the problem introduced in #5627, sorry I'm 
not sure What problem was introduced in #5627? 
   I have tried to reproduce the screenshot menshoned in Issue#5699, but I 
can't reproduce it, it's a little strange.  
   
   + **In addition, I personally think that the current PR introduces a new 
problem: if we create a new user using admin, such as test, after successfully 
creating,  the current logged-in user displayed in the top right corner of the 
page will be updated to the current newly created user,** you can see my 
screenshot based on the latest code of dev:  
   
   1. try to add a new user named test, now the username of current user is 
admin.  
   
   
![image](https://user-images.githubusercontent.com/52202080/124154988-9733c980-dac8-11eb-8587-ad417b31f95e.png)
  
   
   2. after the user has been created,  the username of current user changes 
from admin to test.  
   
   
![image](https://user-images.githubusercontent.com/52202080/124155152-c21e1d80-dac8-11eb-87b9-f228fe991e36.png)
   


-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to