Skip to content
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

Bring crowdinStatsCommand back #9961

Closed
wants to merge 1 commit into from
Closed

Bring crowdinStatsCommand back #9961

wants to merge 1 commit into from

Conversation

sae220
Copy link
Contributor

@sae220 sae220 commented Apr 10, 2024

This command is temporarily disabled by @riknoll in #9954 .
I haven't done any testing because I can't use some API with my key.

@sae220 sae220 closed this Apr 10, 2024
@riknoll
Copy link
Member

riknoll commented Apr 10, 2024

ah, @sae220 I actually just opened a PR fixing this! Check it our here:

#9962

Interesting that it doesn't work with your API key... do you happen to know which API was failing? what permissions did you give it?

@riknoll
Copy link
Member

riknoll commented Apr 10, 2024

Also, were you using this command? If so, can I ask what for? I might be interested in your use case to see if I could make something more comprehensive for you.

@sae220
Copy link
Contributor Author

sae220 commented Apr 10, 2024

Thanks a lot for your great help.

do you happen to know which API was failing? what permissions did you give it?

SorceFiles API (like List Files). This returns 403 Forbidden. I gave all permissons.

Also, were you using this command? If so, can I ask what for?

I only did some test to try to solve #9926. I'm sorry that I can't give you interesting use case.

@riknoll
Copy link
Member

riknoll commented Apr 10, 2024

No worries! Thanks for the PR in any case.

I'll have to investigate if Crowdin has a way to get file IDs without list files, it's a really slow request and the permissions on it do seem weird.

@sae220
Copy link
Contributor Author

sae220 commented Apr 12, 2024

I asked crowdin support team why I can't access some API. The answer is below.

Yes, to access all API capabilities you need to be a manager and have a manager token. Also developer role would work

Now I ask ganicke to give me in crowdin conversations.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants