- 
          
- 
                Notifications
    You must be signed in to change notification settings 
- Fork 442
Added option to update Hugging Face credentials in UI #798
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi,
Seems like you have a merge conflict with our main branch.
Sorry for being delayed on reviewing this. Will try to review asap once you fix this one so there's no more conflicts.
b89d3b8    to
    568c8f6      
    Compare
  
    | Thanks for reviewing. The conflict should be resolved now. | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
568c8f6    to
    f5c9f49      
    Compare
  
    There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| Could you check if the error is still happening? Thanks | 
4acf5a7    to
    d59e80f      
    Compare
  
    d59e80f    to
    c01db3e      
    Compare
  
    There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This works for me, will wait incase @dadmobile wanted to review




Added UI option for updating Hugging Face credentials (Issue #703)
When the user provides a new token, we send it to the backend.
The backend clears the cached Hugging Face token (/model/logout_from_huggingface) before saving the new one.