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

Update BaseConfig.php #2082

Merged
merged 1 commit into from
Aug 20, 2019
Merged

Update BaseConfig.php #2082

merged 1 commit into from
Aug 20, 2019

Conversation

zl59503020
Copy link
Contributor

@zl59503020 zl59503020 commented Jul 2, 2019

Update the initialization configuration class to read the env environment to accommodate multi-level configuration

Description
Update the initialization configuration class to read the env environment to accommodate multi-level configuration

my .env file
website.rpc.app.mall = http://192.168.1.3:831/rpc/manage/mall/init_rpc_server

and my WebSite config class

class WebSite extends BaseConfig
{
public $rpc = [
      'app' => [
          'mall' => 'http://domain/rpc/manage/mall/init_rpc_server', 
      ],
];
}

Checklist:

  • Securely signed commits
  • Component(s) with PHPdocs
  • Unit testing, with >80% coverage
  • User guide updated
  • Conforms to style guide

Update the initialization configuration class to read the env environment to accommodate multi-level configuration
@lonnieezell lonnieezell merged commit d81330a into codeigniter4:develop Aug 20, 2019
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