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.

Sign up for GitHub

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

Jump to bottom

env variables are moved to json config file #172

Merged
merged 25 commits into from
Jan 3, 2023
Merged

env variables are moved to json config file #172

merged 25 commits into from
Jan 3, 2023

Conversation

ikaratass
Copy link
Collaborator

@ikaratass ikaratass commented Dec 1, 2022 •
edited
Loading

  • network interface is still configured in the env file to run the simulator according to secc settings.
  • Energytransfermode could be set
  • certificate installation required flag can be configured.
  • For each configuration can be found under the example folder.
  • The items below have moved to the JSON config file.
    supported_protocols
    supported_energy_services
    is_cert_install_needed
    use_tls
    sdp_retry_cycles
    max_contract_certs
    enforce_tls
    max_supporting_points

@ikaratass ikaratass force-pushed the feat/ev_config branch 2 times, most recently from ca639ae to 79d6f30 Compare December 16, 2022 09:19
@shalinnijel2 shalinnijel2 mentioned this pull request Dec 16, 2022
Closed
@ikaratass ikaratass marked this pull request as ready for review December 16, 2022 10:55
shalinnijel2
shalinnijel2 suggested changes Dec 20, 2022
View reviewed changes
README.md Outdated Show resolved Hide resolved
iso15118/evcc/evcc_config.py Outdated Show resolved Hide resolved
iso15118/shared/utils.py Outdated Show resolved Hide resolved
iso15118/shared/utils.py Show resolved Hide resolved
iso15118/shared/utils.py Outdated Show resolved Hide resolved
shalinnijel2
shalinnijel2 suggested changes Dec 22, 2022
View reviewed changes
iso15118/evcc/evcc_config.py Outdated Show resolved Hide resolved
@ikaratass ikaratass merged commit 5d86090 into master Jan 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@shalinnijel2 shalinnijel2 shalinnijel2 approved these changes

@tropxy tropxy Awaiting requested review from tropxy

@mdwcrft mdwcrft Awaiting requested review from mdwcrft

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ikaratass @shalinnijel2