Push: Add "msg" as encoding

[rhuss]

"msg" is the default encoding anyway, but for consistency's sake, we should add it as a possible option for the encoding: configuration field.

[rhuss]

I already documented this in https://github.com/evcc-io/docs/pull/675. It felt unnatural to leave out a dedicated option for having the msg only and rely only on the default.

[andig]

Even if documented, adding this diesn‘t do anything. Might as well leave out, docs are still correct.

[rhuss]

"Does it hurt?" 😄

@andig From a UI perspective, it's still a difference if you have to leave something out to achieve a certain behaviour that you can't achieve otherwise. If you can specify "title" as an option, for me as a user, it is a surprise that there is no "msg", too, but that I have to rely on the default.

Aber egal, alles gut. Ich pendel mich schon noch ein :-)

[andig]

From a UI perspective

UI hat hiermit nichts zu tun, bitte nicht immer alles in einen Topf werfen!

[andig]

PS.: magst Du Dich mal bei info@evcc.io oder auf Slack melden?

[rhuss]

Configfile schemata sind auch teil eines user interfaces (UI), oder taeusche ich mich da ? Genauso wie CLI optionen UI sind. Alles was mit dem "user" eine Schnittstelle hat ist UI, egal in welchen Rolle (admin oder enduser).

[andig]

Configfile schemata sind auch teil eines user interfaces (UI), oder taeusche ich mich da ? Genauso wie CLI optionen UI sind

…und nichts davon passiert in diesem PR.

[rhuss]

Sorry, my bad 🤦 . Da war ein Fehler im PR, ich meinte:

    case "msg":
        res = msg
    default:
        m.log.ERROR.Printf("Invalid enconding: %s (should be one of 'msg', 'title', 'json', 'csv' or 'tsv')", m.encoding)
        return

Ging neben der Symmetrie auch darum Typos ala jason oder, ganz beliebt :-), cvs, zu erkennen.