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

Internationalization #574

Closed
shalvah opened this issue Dec 1, 2022 · 14 comments
Closed

Internationalization #574

shalvah opened this issue Dec 1, 2022 · 14 comments

Comments

@shalvah
Copy link
Contributor

shalvah commented Dec 1, 2022

No description provided.

@vpratfr
Copy link
Contributor

vpratfr commented Jan 17, 2023

I would be interested in a way to generate it in French too. If needed I can even make a PR to provide FR translation later on.

@guleswine
Copy link

I was disappointed to find that the authorization text is hardwired into this file and not even included in the localization packs vendor/knuckleswtf/scribe/src/Extracting/ApiDetails.php 😳

@vpratfr
Copy link
Contributor

vpratfr commented Apr 20, 2023

@guleswine do not hesitate to make a PR if you have time. On our side, we decided to document the API in English so this is not an issue anymore for us.

@guleswine
Copy link

guleswine commented Apr 20, 2023

@guleswine do not hesitate to make a PR if you have time. On our side, we decided to document the API in English so this is not an issue anymore for us.

I think it's not even just a localization problem, I think this problem is more serious, because you can't edit the text without changing the important package file

@vpratfr
Copy link
Contributor

vpratfr commented Apr 20, 2023

You can change those files too in your PR, no problem.

@guleswine
Copy link

guleswine commented Apr 20, 2023

You can change those files too in your PR, no problem.

This is great, but don't forget the main thing when a person is looking for a tool for auto-generating documentation, he wants to save development time, not increase it.

first of all, I wanted to support this topic and point out the problem, code organization, storing text in classes is a dubious idea

if this does not become a critical problem for us, and we will use it, I will be happy to do PR

@vpratfr
Copy link
Contributor

vpratfr commented Apr 20, 2023

I am not the maintainer of this project, but don't forget too that the main thing with free open-source libraries, is that it is not a product sold by @shalvah and that the community is also supposed to contribute (you already solve many hours by using it, even if not perfect).

@guleswine
Copy link

guleswine commented Apr 20, 2023

I am not the maintainer of this project, but don't forget too that the main thing with free open-source libraries, is that it is not a product sold by @shalvah and that the community is also supposed to contribute (you already solve many hours by using it, even if not perfect).

I didn’t say that I use it, and I said that for a start I just wanted to support this topic, I understand perfectly well about PR, but for some reason you repeat the same thing...
And in my next messages I just explained to you that this is a problem of organizing the code in this way, more seriously than you see it

P.S. judging by this answer, it may not be advisable to make a PR in this project🙂
#589 (comment)

@shalvah
Copy link
Contributor Author

shalvah commented Apr 20, 2023

P.S. judging by this answer, it may not be advisable to make a PR in this projec

Lol, wtf man. That's an excuse. You can literally click on the Pull Requests tab to see 20+ external PRs I've merged this year.

No idea how you picked an issue where I'm complaining about people bugging me about an unimportant issue with another package, and using that to justify not contributing. Did you even read that issue?🤨 FFS, this issue was created by me. Doesn't that mean I think it's important?

I respond to issues and PRs promptly when I can, so I really don't get you. If you have a problem, talk to me. Life happens. You have other shit you're doing, and so do I. I'll get to this when I can. If you want to help, ask how. Otherwise, you'll have to wait until I can find a solution. This whole thing just sounds ungrateful.

@shalvah
Copy link
Contributor Author

shalvah commented Apr 20, 2023

I just explained to you that this is a problem of organizing the code in this way, more seriously than you see it

True, it's a problem. But you need to remember that software development has to follow priorities. What's a serious problem to one person doesn't matter to another. No software is perfect. And over time, priorities will adjust and we will fix some problems (and probably add new ones). This package has evolved over the last few years to allow more end user flexibility while maintaining simplicity, and this will be another step in that.

@guleswine
Copy link

guleswine commented Apr 20, 2023

@shalvah I understand you, no problem bro, just my first look at the project and a small dispute with another user😉

Your answer pleases, in that case, so, then a question for a potential PR, do you think it's right to put text pieces into laravel language packs? Because I see that you have taken out some parts of the text description in the config file

@shalvah
Copy link
Contributor Author

shalvah commented Apr 20, 2023

I obviously think it's a problem. That's why I created this issue. See also #639 (comment) Progress happens little by little.

@shalvah
Copy link
Contributor Author

shalvah commented Apr 21, 2023

You'll be pleased to know that it's underway: #647

@shalvah
Copy link
Contributor Author

shalvah commented Apr 22, 2023

Released in 4.19.0

@shalvah shalvah closed this as completed Apr 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants