Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Use /models/ path for api call to update settings #1049

Merged
merged 3 commits into from
Sep 14, 2022

Conversation

Wauplin
Copy link
Contributor

@Wauplin Wauplin commented Sep 14, 2022

Related to https://github.com/huggingface/moon-landing/pull/3796#discussion_r970751649 (internal url).

Use updated api route for consistency
Previously: ".../api/user/repo_name/settings" and ".../api/datasets/user/repo_name/settings"
Now: ".../api/models/user/repo_name/settings" and ".../api/datasets/user/repo_name/settings"

@HuggingFaceDocBuilderDev
Copy link

HuggingFaceDocBuilderDev commented Sep 14, 2022

The documentation is not available anymore as the PR was closed or merged.

@codecov
Copy link

codecov bot commented Sep 14, 2022

Codecov Report

Merging #1049 (63d474f) into main (b60bdd6) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1049      +/-   ##
==========================================
- Coverage   83.80%   83.79%   -0.01%     
==========================================
  Files          37       37              
  Lines        3957     3956       -1     
==========================================
- Hits         3316     3315       -1     
  Misses        641      641              
Impacted Files Coverage Δ
src/huggingface_hub/lfs.py 74.84% <ø> (ø)
src/huggingface_hub/hf_api.py 87.44% <100.00%> (-0.02%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@Wauplin Wauplin merged commit 7dee38c into main Sep 14, 2022
@Wauplin Wauplin deleted the simplify-api-call-in-update-repo-visibility branch September 14, 2022 13:32
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants