rmorawski/standardnotes-app-web
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

chore: dont show alert on every role change

Browse Source
This commit is contained in:
Aman Harwara
2023-09-04 21:13:42 +05:30
parent 4ee5ad8595
commit c3890fe50d
1 changed files with 11 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
packages/snjs/lib/Services/Features/FeaturesService.ts
View File

@@ -360,7 +360,8 @@ export class FeaturesService
const isInitialLoadRolesChange = previousRoles.length === 0 const isInitialLoadRolesChange = previousRoles.length === 0
if (!isInitialLoadRolesChange) { if (!isInitialLoadRolesChange) {
if (this.onlineRolesIncludePaidSubscription()) { const didPreviousRolesIncludePaidSubscription = this.rolesIncludePaidSubscription(previousRoles)
if (this.onlineRolesIncludePaidSubscription() && !didPreviousRolesIncludePaidSubscription) {
await this.notifyEvent(FeaturesEvent.DidPurchaseSubscription) await this.notifyEvent(FeaturesEvent.DidPurchaseSubscription)
} }
} }
@@ -395,10 +396,16 @@ export class FeaturesService
return !isNativeFeature return !isNativeFeature
} }
onlineRolesIncludePaidSubscription(): boolean { private rolesIncludePaidSubscription(roles: string[]) {
const unpaidRoles = [RoleName.NAMES.CoreUser] {
const unpaidRoles = [RoleName.NAMES.CoreUser]
return this.onlineRoles.some((role) => !unpaidRoles.includes(role)) return roles.some((role) => !unpaidRoles.includes(role))
}
}
onlineRolesIncludePaidSubscription(): boolean {
return this.rolesIncludePaidSubscription(this.onlineRoles)
} }
public rolesBySorting(roles: string[]): string[] { public rolesBySorting(roles: string[]): string[] {