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

Twitter native test : Locale and Timezone #2564

Open
zbendhiba opened this issue May 7, 2021 · 4 comments
Open

Twitter native test : Locale and Timezone #2564

zbendhiba opened this issue May 7, 2021 · 4 comments
Labels
bug Something isn't working good first issue Good for newcomers

Comments

@zbendhiba
Copy link
Contributor

I've been testing the twitter extension in native mode. And as I have a Locale fr, I've got errors parsing the responses from Twitter.com.
The Twitter Java API expect to have the Locale.US installed and expect also Timezone UTC.
We need to address the issue or at least document it.

@zbendhiba zbendhiba added the bug Something isn't working label May 7, 2021
@zbendhiba
Copy link
Contributor Author

related to camel-quarkus-1861 and quarkus-5244

@ffang
Copy link
Contributor

ffang commented May 9, 2021

Hi @zbendhiba ,

About Locale, Hi @zbendhiba ,

Just FYI, if it's only one no-default locale needed in native mode, we can specify it in application.properties like

quarkus.native.user-country=US
quarkus.native.user-language=en

Please take a look at #2550 (comment) to get more details.

About the timezone, please take a look at my way to fix #2542

Best Regards
Freeman

@zbendhiba
Copy link
Contributor Author

thanks @ffang

The quarkus.native.user-country, and quarkus.native.user-language work fine for me. From a user point of view, I hope, we can define more than one locale in near future.

For the timeZone, we don't use docker for twitter API. There is fix here. I wanted to ask you to test Twitter integration tests to check if there is problem with TimeZone, but it requires developer credentials.

@zbendhiba
Copy link
Contributor Author

If someone could document this one and add the country/Language parameters to make this native build work for everyone.

@zbendhiba zbendhiba added the good first issue Good for newcomers label Apr 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants