review-fixes

_locales/en/messages.json

@@ -2377,6 +2377,10 @@
   "displayName": {
     "message": "Display Name"
   },
+  "anonymous": {
+    "message": "Anonymous",
+    "description": "The name of currently unidentified users"
+  },
   "enterDisplayName": {
     "message": "Enter a display name"
   },

js/background.js

@@ -1093,7 +1093,9 @@
     // Attempts a connection to an open group server
     window.attemptConnection = async (serverURL, channelId) => {
       let completeServerURL = serverURL.toLowerCase();
-      const valid = window.libsession.Types.OpenGroup.validate(completeServerURL);
+      const valid = window.libsession.Types.OpenGroup.validate(
+        completeServerURL
+      );
       if (!valid) {
         return new Promise((_resolve, reject) => {
           reject(window.i18n('connectToServerFail'));
@@ -1101,7 +1103,9 @@
       }
 
       // Add http or https prefix to server
-      completeServerURL = window.libsession.Types.OpenGroup.prefixify(completeServerURL);
+      completeServerURL = window.libsession.Types.OpenGroup.prefixify(
+        completeServerURL
+      );
 
       const rawServerURL = serverURL
         .replace(/^https?:\/\//i, '')

ts/components/session/LeftPaneMessageSection.tsx

@@ -475,36 +475,34 @@ export class LeftPaneMessageSection extends React.Component<Props, State> {
     }
 
     // Connect to server
-    const successPromise = OpenGroup.join(serverUrl, async () => {
-      if (await OpenGroup.serverExists(serverUrl)) {
-        window.pushToast({
-          title: window.i18n('connectingToServer'),
-          id: 'connectToServerSuccess',
-          type: 'success',
-        });
-
-        this.setState({ loading: true });
-      }
-    });
+    try {
+      await OpenGroup.join(serverUrl, async () => {
+        if (await OpenGroup.serverExists(serverUrl)) {
+          window.pushToast({
+            title: window.i18n('connectingToServer'),
+            id: 'connectToServerSuccess',
+            type: 'success',
+          });
+
+          this.setState({ loading: true });
+        }
+      });
 
-    successPromise
-      .then(() => {
-        this.handleToggleOverlay(undefined);
-        this.setState({
-          loading: false,
-        });
-      })
-      .catch(() => {
-        this.setState({
-          loading: false,
-        });
+      this.handleToggleOverlay(undefined);
+      this.setState({
+        loading: false,
+      });
+    } catch (e) {
+      this.setState({
+        loading: false,
+      });
 
-        window.pushToast({
-          title: window.i18n('connectToServerFail'),
-          id: 'connectToServerFail',
-          type: 'error',
-        });
+      window.pushToast({
+        title: window.i18n('connectToServerFail'),
+        id: 'connectToServerFail',
+        type: 'error',
       });
+    }
   }
 
   private async onCreateClosedGroup(

ts/components/session/SessionClosableOverlay.tsx

@@ -69,7 +69,7 @@ export class SessionClosableOverlay extends React.Component<Props, State> {
     const contactsList = this.props.contacts ?? [];
 
     return contactsList.map((d: any) => {
-      const name = d.name ?? 'Anonymous';
+      const name = d.name ?? window.i18n('anonymous');
 
       // TODO: should take existing members into account
       const existingMember = false;

ts/session/types/OpenGroup.ts

@@ -1,3 +1,5 @@
+import { ConversationModel } from '../../../js/models/conversations';
+
 interface OpenGroupParams {
   server: string;
   channel: number;
@@ -144,7 +146,9 @@ export class OpenGroup {
    * @param server The server URL
    * @returns BackBone conversation model corresponding to the server if it exists, otherwise `undefined`
    */
-  public static async getConversation(server: string): Promise<any> {
+  public static async getConversation(
+    server: string
+  ): Promise<ConversationModel | undefined> {
     if (!OpenGroup.validate(server)) {
       return;
     }
@@ -161,17 +165,6 @@ export class OpenGroup {
     return serverInfo.channels[0].conversation;
   }
 
-  /**
-   * Get the conversation model of a server from conversation ID
-   *
-   * @param conversationId The server's conversation ID
-   * @returns BackBone conversation model corresponding to the server if it exists, otherwise `undefined`
-   */
-  public static getConversationByCID(conversationId: string): any {
-    const { ConversationController } = window;
-    return ConversationController.get(conversationId);
-  }
-
   /**
    * Check if the server exists.
    * This does not compare against your conversations with the server.