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

Warning no default phone region set #1465

Closed
m0rtalis opened this issue Apr 5, 2021 · 5 comments
Closed

Warning no default phone region set #1465

m0rtalis opened this issue Apr 5, 2021 · 5 comments
Labels
enhancement feature: auto config (environment variables) Auto configuring via environment variables

Comments

@m0rtalis
Copy link
Contributor

m0rtalis commented Apr 5, 2021

With nextcloud 21.0 there is a new warning saying

Your installation has no default phone region set. This is required to validate phone numbers in the profile settings without a country code. To allow numbers without a country code, please add "default_phone_region" with the respective ISO 3166-1 code of the region to your config file.

It was briefly discussed in issue #1414 this setting should be settable with an environment variable. Currently it seems like I have to edit the config.php just for this one setting.

@nathanael-h
Copy link

Hello, can someone confirm or not if to solve this we need to add this env variable in autoconfig.php files (eg : https://github.com/nextcloud/docker/blob/master/21.0/apache/config/autoconfig.php ) ?

@tmo1
Copy link

tmo1 commented Jan 12, 2022

It would indeed be nice to allow this to be set via an environmental variable, but it can be set from the command line like this:

# docker-compose exec --user www-data <app> php occ config:system:set default_phone_region --value="US"

where <app> is the name of the nextcloud service in the nextcloud docker-compose.yml.

I found this in this thread.

@joshtrichards
Copy link
Member

Also see #1544 (comment)

I doubt an additional environment variable just for Docker will be added for this. THE NC_* variables are a sufficient workaround for those that need it to be entirely in their Compose file IMO.

For the most part, I think we should nudge Nextcloud Server - rather than the community Docker image - towards more extensive and robust NC_* support - e.g. nextcloud/documentation#9143 (comment)

This is mostly an upstream (NC Server) matter. This issue should probably get closed out here.

@tigerblue77
Copy link

tigerblue77 commented Jul 23, 2023

@joshtrichards I think TZ environment variable is a good practice when it comes to docker containers
Maybe it could be based on it ?

@joshtrichards joshtrichards added the feature: auto config (environment variables) Auto configuring via environment variables label Oct 25, 2023
@joshtrichards
Copy link
Member

joshtrichards commented Nov 4, 2023

Even though a later issue was closed as a duplicate of this one, the discussion was more extensive there after it was closed. If you want to see this, use the NC_* variable noted: #1544 (comment)

Also see discussion in #1795 (same solution).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement feature: auto config (environment variables) Auto configuring via environment variables
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants