Open-Assistant icon indicating copy to clipboard operation
Open-Assistant copied to clipboard

Admin interface: Change display name (#2849)

Open DeanChugall opened this issue 1 year ago • 0 comments

Hello 👋,

I hope you're doing great! I'm delighted to submit this pull request for enhancing the admin interface #2849 by adding the ability to change the display name of users.

🌟 Changes Made

Below is a summary of the changes I've made in this PR:

FRONTEND:


  • [id].tsx - remove isDisabled from FormControl.
  • oasst_api_client.ts - change API PUT url in set_user_status method to accept display_name parameter.
  • update_user.tsx add display_name parameter.

BACKEND:


  • user.py -> update update_user function signature *(added display_name: Optional[str] = None)
  • user_repository.py update update_user function signature *(added display_name: Optional[str] = None) and add check in if statment.

🤝 Your Feedback is Valued

I appreciate your time in reviewing this PR, and I'm looking forward to any feedback or suggestions you might have. If you have any questions or need further clarification, please don't hesitate to reach out. I'll be more than happy to assist.

Thank you once again for considering this contribution to the project! Let's continue working together to make our admin interface even better! 😊

Best regards,

datatab.

DeanChugall avatar Apr 25 '23 17:04 DeanChugall