Browse Source

Fix last seen & spacing issue (#3567)

* Fix last seen & spacing issue

* Address PR comments, moving try catch to config service
pull/3633/head
Colton Hurst 3 years ago committed by GitHub
parent
commit
d168d5ee9b
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
  1. 2
      apps/browser/src/_locales/en/messages.json
  2. 6
      apps/browser/src/popup/settings/about.component.html
  3. 18
      libs/common/src/services/config/config.service.ts

2
apps/browser/src/_locales/en/messages.json

@ -2017,7 +2017,7 @@ @@ -2017,7 +2017,7 @@
}
},
"lastSeenOn": {
"message": "last seen on $DATE$",
"message": "last seen on: $DATE$",
"placeholders": {
"date": {
"content": "$1",

6
apps/browser/src/popup/settings/about.component.html

@ -14,7 +14,7 @@ @@ -14,7 +14,7 @@
<p class="text-center" *ngIf="isCloud">
{{ "serverVersion" | i18n }}: {{ this.serverConfig?.version }}
<span *ngIf="!serverConfig.isValid()">
({{ "lastSeenOn" | i18n }}: {{ serverConfig.utcDate | date: "mediumDate" }})
({{ "lastSeenOn" | i18n: (serverConfig.utcDate | date: "mediumDate") }})
</span>
</p>
@ -24,7 +24,7 @@ @@ -24,7 +24,7 @@
{{ "serverVersion" | i18n }} <small>({{ "thirdParty" | i18n }})</small>:
{{ this.serverConfig?.version }}
<span *ngIf="!serverConfig.isValid()">
({{ "lastSeenOn" | i18n }}: {{ serverConfig.utcDate | date: "mediumDate" }})
({{ "lastSeenOn" | i18n: (serverConfig.utcDate | date: "mediumDate") }})
</span>
</p>
<div class="text-center">
@ -36,7 +36,7 @@ @@ -36,7 +36,7 @@
{{ "serverVersion" | i18n }} <small>({{ "selfHosted" | i18n }})</small>:
{{ this.serverConfig?.version }}
<span *ngIf="!serverConfig.isValid()">
({{ "lastSeenOn" | i18n }}: {{ serverConfig.utcDate | date: "mediumDate" }})
({{ "lastSeenOn" | i18n: (serverConfig.utcDate | date: "mediumDate") }})
</span>
</p>
</ng-container>

18
libs/common/src/services/config/config.service.ts

@ -48,14 +48,16 @@ export class ConfigService implements ConfigServiceAbstraction { @@ -48,14 +48,16 @@ export class ConfigService implements ConfigServiceAbstraction {
}
private async fetchServerConfig(): Promise<ServerConfig> {
const response = await this.configApiService.get();
const data = new ServerConfigData(response);
if (data != null) {
await this.stateService.setServerConfig(data);
return new ServerConfig(data);
try {
const response = await this.configApiService.get();
if (response != null) {
const data = new ServerConfigData(response);
await this.stateService.setServerConfig(data);
return new ServerConfig(data);
}
} catch {
return null;
}
return null;
}
}

Loading…
Cancel
Save