SAP / open-ux-tools

Enable community collaboration to jointly promote and facilitate best in class tooling capabilities
Apache License 2.0
83 stars 35 forks source link

TBI - Convert ux-store `AuthenticationType` to string literal type as ux-store is importing keytar #2049

Closed IainSAP closed 3 months ago

IainSAP commented 3 months ago

Description (include screenshots)

Imports only requiring the AuthenticationType from @sap-ux/store are also importing its dependencies (e.g keytar) since this is an enum. We should replace with a real type (string literal) and move to dev dependencies (@sap-ux/ui5-config)

https://github.com/SAP/open-ux-tools/blob/main/packages/ui5-config/src/types/middlewares.ts#L1

Value

Prevents issues with bundling when keytar is included.

Architecture Elaboration

Does it requires architecture elaboration?

Notes

Tasks

IainSAP commented 3 months ago

Done