Closed bedhub closed 2 years ago
another point is enums: https://www.typescriptlang.org/docs/handbook/enums.html
I would recommend to use as const
objects at first or maybe const enum
where necessary because they have the lowest overhead.
Some preliminary client testing revealed the following issues:
mithril.js:1233 TypeError: Vnodes must either always have keys or never have keys!
at Function.Vnode.normalizeChildren (mithril.js:21:11)
at Function.Vnode.normalize (mithril.js:7:73)
at Function.Vnode.normalizeChildren (mithril.js:25:25)
at hyperscript (mithril.js:148:26)
at m (mithril.js:1495:43)
at SidebarSection.view (SidebarSection.ts:18:10)
at Function.callHook (mithril.js:308:16)
at initComponent (mithril.js:427:46)
at createComponent (mithril.js:432:3)
at createNode (mithril.js:343:8)
@nowhub and I are testing Contacts.
@nowhub and I are testing Appearance settings.
@nowhub and I are testing Whitelabel settings.
Found the following minor problem:
The same will happen always if the accountType is something other than Premium. It happens because of a change in TutanotaConstants.ts
:
export const AccountTypeNames = ["System", "Free", "Outlook", "Premium", "Stream", "External"]
was replaced with
export const AccountTypeNames = reverse(AccountType)
We should either keept the reversed map explicitly or convert the enum to natural capitalization for the definition of AccountTypeNames
.
I will test subscription settings.
@nowhub and I are testing Contact form settings.
@nowhub and I are testing Global settings.
I am testing PaymentSettings
I am testing GroupSettings
I am testing UserSettings.
I will continue with MailSettings now.
I will focus on search now
will start testing mail editing, now
@vaf-hub can you test the rejected senders list? I cannot seem to make anything appear there, don't know what the correct conditions would be. Nvm found the server config.
I'll test mail folders and drag & drop now.
We want to migrate our typesystem from flow to TypeScript.
Test notes
/signup#subscription=premium&type=private&interval=12
preselects the option and skips the step