add getCustomerId to the interface #19
Conversation
art333
requested review from
segfault79,
Tooobi,
fgerardwltn24,
bwagneras,
daniel-krug and
andrespringt
src/ps-web-apis.ts
Outdated
| /** | ||
| * will request customer pseudo id for currently logged user from consent backend | ||
| */ | ||
| getCustomerId(additionalParameter?: Map<string, string[]>): Promise<string>; |
There was a problem hiding this comment.
mich wundert nur dass es hier keine spezifischen parameter gibt... welche additional parameters gibt es denn?
das typing soll ja eigtl. helfen wenn man das modul nutzt. so frag ich mich was ich da mitgeben kann und warum
There was a problem hiding this comment.
"clientId" ist so ein pflicht parameter. Ich dachte ich mache mir alle parameters als additional ^^ aber ok ich habe jetzt noch "clientId: string" da hinzugefügt
There was a problem hiding this comment.
aber was sind denn die additional parameters? ... das ist so als ob du in java zu Object castest und das ist dann die API
There was a problem hiding this comment.
Schonmal generisch für alle alle kommenden Parameter vorbereitet? ;)
There was a problem hiding this comment.
Ja ich dachte erst dass wir es generisch machen wollen wie bei doLogin, doRegister und so :D aber egal ich habe nur das "clientId" da gelasen. Erst sollte es passen :D
No description provided.