Merge pull request #620 from nextcloud/feat/workaround-for-multi-account

feat: workaround for multi-account support with several instance with different executable names

Author: ShGKme

README.md

@@ -21,6 +21,26 @@
 - Works with limitations:
   - File Viewer — only images and videos
 
+## 👥 Multi-account
+
+Full multi-account currently [is not currently supported](https://github.com/nextcloud/talk-desktop/issues/7).
+
+However, using portable `zip` distribution, you can have several Nextcloud Talk instances run simultaneously. Just rename the executable from default  `Nextcloud Talk` to a custom name. For example: 
+
+```
+/path/to/apps/
+├── home-apps/
+│   └── Nextcloud Talk/
+│       ├── ...
+│       ├── Nextcloud Talk (Home).exe
+│       └── ...
+└── work-apps/
+    └── Nextcloud Talk/
+        ├── ...
+        ├── Nextcloud Talk (Work).exe
+        └── ...
+```
+
 ## 🧑‍💻 Development Setup
 
 ### Initial setup

src/main.js

@@ -42,12 +42,12 @@ const ARGUMENTS = {
 }
 
 /**
- * Separate production and development instances, including application and user data
+ * On production use executable name as application name to allow several independent application instances.
+ * On development use "Nextcloud Talk (dev)" instead of the default "electron".
  */
-if (process.env.NODE_ENV === 'development') {
-	app.setName('Nextcloud Talk (dev)')
-	app.setPath('userData', path.join(app.getPath('appData'), 'Nextcloud Talk (dev)'))
-}
+const APP_NAME = process.env.NODE_ENV !== 'development' ? path.parse(app.getPath('exe')).name : 'Nextcloud Talk (dev)'
+app.setName(APP_NAME)
+app.setPath('userData', path.join(app.getPath('appData'), app.getName()))
 app.setAppUserModelId(app.getName())
 
 /**
@@ -71,6 +71,7 @@ if (process.env.NODE_ENV === 'production') {
 
 ipcMain.on('app:quit', () => app.quit())
 ipcMain.handle('app:getOs', () => getOs())
+ipcMain.handle('app:getAppName', () => app.getName())
 ipcMain.handle('app:getSystemL10n', () => ({
 	locale: app.getLocale().replace('-', '_'),
 	language: app.getPreferredSystemLanguages()[0].replace('-', '_'),

src/preload.js

@@ -48,6 +48,12 @@ const TALK_DESKTOP = {
 	 * @type {typeof packageInfo} packageInfo
 	 */
 	packageInfo,
+	/**
+	 * Get app name
+	 *
+	 * @return {Promise<string>}
+	 */
+	getAppName: () => ipcRenderer.invoke('app:getAppName'),
 	/**
 	 * Quit the application
 	 */

src/shared/setupWebPage.js

@@ -84,13 +84,15 @@ async function applyL10n() {
 
 /**
  * Make all required initial setup for the web page:
+ * - set title according to app name
  * - restore app data
  * - get OS info
  * - apply theme to HTML data-attrs
  * - apply locale to HTML lang and data-locale attrs
  * - register translation bundles for Talk and Talk Desktop
  */
 export async function setupWebPage() {
+	document.title = await window.TALK_DESKTOP.getAppName()
 	initGlobals()
 	appData.restore()
 	window.OS = await window.TALK_DESKTOP.getOs()