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

I18n en translations reference #425

Merged
merged 30 commits into from
Sep 13, 2023

Conversation

carygravel
Copy link
Contributor

@carygravel carygravel commented Aug 13, 2023

This is still a work in progress. Just wanted to be transparent about how far I had got. Note that I started with @duckfullstop 's branch. If his commits are a problem, I can rebase/remove them.

@duckfullstop
Copy link
Contributor

Could you rebase onto main without my commits? Should keep things a little tidier, especially as you're touching a section of the documentation that I haven't.

@carygravel
Copy link
Contributor Author

carygravel commented Aug 14, 2023

Just removed them. Somehow the diff to main affects some files I didn't touch. Can investigate again later if necessary.

@carygravel carygravel force-pushed the i18n-en-translations-reference branch from aaef6e5 to 54374d3 Compare August 14, 2023 08:05
@carygravel
Copy link
Contributor Author

That's reference finished, I think. Please take a look!

Copy link
Contributor

@duckfullstop duckfullstop left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM pending comment on the previous stuff

allows us to calculate at any point in time the surplus power. Measuring the
generated power is interesting, but has no effect on the function, with the
this exception:
https://docs.evcc.io/docs/guides/meters#ich-habe-eine-pv-anlage-aber-keinen-netzanschluss-z%C3%A4hler-kann-ich-evcc-trotzdem-nutzen
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this be a link to the EN documentation?


The minimum power required to charge is calculated from the values `minCurrent`
and `phases`, defined per `loadpoint` (a group of colocated chargers)
See https://docs.evcc.io/docs/reference/configuration/loadpoints for more
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same here (and in other places)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, this should definitly be a relative link like this:

Suggested change
See https://docs.evcc.io/docs/reference/configuration/loadpoints for more
See [`loadpoints`](./loadpoints)

Comment on lines +7 to +10
evcc requires a configuration file in which the system is described. Without
this file, evcc cannot be used. The file itself is written in
[YAML](https://en.wikipedia.org/wiki/YAML). This format defines a syntax with
which data can be structured in plain text.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could this be rephrased to something that flows a little more naturally? I feel like this might be a bit too direct a translation from DE.

@naltatis
Copy link
Member

Looks very nice! Do you have anything beside the linking comments open? Otherwise I would merge this.

@evcc-io evcc-io deleted a comment from vercel bot Sep 10, 2023
@duckfullstop
Copy link
Contributor

@carygravel Any idea what state this is in at the moment? Could just merge this and then tackle the links in a separate issue

@naltatis
Copy link
Member

@carygravel As suggested by @duckfullstop, I'll merge this now and we can update in smaller increments from there.

@naltatis naltatis merged commit 43ec646 into evcc-io:main Sep 13, 2023
@carygravel
Copy link
Contributor Author

Apologies for the lack of response. Things are a little fraught at the moment. I'll fix the comments ASAP.

@carygravel carygravel deleted the i18n-en-translations-reference branch September 13, 2023 17:58
naltatis pushed a commit that referenced this pull request Sep 24, 2023
* i18n: correct links from EN to DE

* i18n: make EN links relative

* i18n: rephrase config introduction paragraph
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.

3 participants