Upgrade Electron to version 24, node to 18 and npm to 9. Electron changed to using node 18 in 23, with node 18 using npm 9 as default.
There doesn't seem to be any breaking changes except the deprecation of Windows 7, 8.1, and Server 2012. A somewhat undocumented breaking change was that elements in the title bar are now draggable which broke the account switching selector. Resolved by adding a no-drag css rule.
- electronjs.org/blog/electron-22-0
- electronjs.org/blog/electron-23-0
- electronjs.org/blog/electron-24-0
The importers recently got extracted from libs/common. As the import functionality is currently limited to the web and cli, there is no need to build browser and desktop as there won't be any changes.
This will free up some runners for other workflows (GH actions) to get faster builds that actually impact browser and desktop.
Another benefit is faster feedback on the PR-checks for those two
* Added new workflow
* Fixed quotes.
* Added step to job.
* Added check for PR destination branch.
* Changed criteria.
* Added fork check for community PRs.
* Removed space
* Changed name.
* Updated with more filters.
* Changed to target approval for adding the label.
* Lint
* Refactored to use pull_request_target instead.
* Fixed comment
* pull_request_target isn't supported, so pull_request instead
* Added comment about target branch.
* Remove cloud web container build. Add QA ACR back in temporarily
* Move selfhost build job into the matrix
* bypassing correct image generation
* fixing linting issues
* Fix output name
* Patch DockerHub upload from server to clients
* Add script to export sha from latest.yaml to files
* Comment out for testing
* Comment out
* Fix
* Remove test comments
* Use new gh action
* Change get checksums action reference to commit hash from master
* Comment out for testing
* Change get-checksum version to branch fix-get-checksum-action
* Comment out gh release if
* Restore if in create release step
* TEst
* Remove comments
* Change path
* Remove unused script
* Add commit SHA for get checksum action
* Update .github/workflows/release-desktop.yml
Co-authored-by: Vince Grassia <593223+vgrassia@users.noreply.github.com>
---------
Co-authored-by: Vince Grassia <593223+vgrassia@users.noreply.github.com>
* Update the QA deploy workflow to deploy to multiple non-prod environments
* Fix linter issues
* Update with testing code if run with QA
* Fix bash typo
* revert name
* Fix name
* Adding some logging
* Fix environment branch typo
* Add QA configuration and renable
* Rename workflow file
* Update .github/workflows/deploy-non-prod-web.yml
Switch config logic to a more extensible one.
Co-authored-by: mimartin12 <77340197+mimartin12@users.noreply.github.com>
* Fix a typo
* Add the original workflow in for QA for no disruptions
* Add line
---------
Co-authored-by: mimartin12 <77340197+mimartin12@users.noreply.github.com>
* pkg ref argon2
* add argon to cli package.json
* argon2 assets
* ci matrix for cli
* simplify builds for testing
* fix env
* zip for each os
* add os prefix back to zip
* create checksums
* make sums 256
* fix shasum command
* combine some steps for unix
* LOWER_RUNNER_OS
* SHORT_RUNNER_OS
* SHORT_RUNNER_OS on dist
* upload unix assets
* restore old build files
* fix formatting
* fix zip folder structure
* fix condition on upload unix zip asset
* compute shasum like other parts of build
* change matrix os targets
* Add build command for ee
* Add config for ee
* Add build workflow for ee
* Change build pipeline for web ee
* Fix name of workflow in trigger
* Build ee image in web build workflow
* Fix name on matrix
* FIx name of the artifact
* Comment out zip extract
* Add zip extract
* All listing dir before unzip
* Add pwd
* Comment out unzipping
* Fix
* Add matrix instead of two stages
* Remove build web ee workflow
* Fix name
* Add needs QA to enforce labels
* Update .github/workflows/enforce-labels.yml
Co-authored-by: Daniel James Smith <djsmith85@users.noreply.github.com>
Co-authored-by: Daniel James Smith <djsmith85@users.noreply.github.com>
* Add tests step to build pipelines
* RUn tests for safari
* Remove tests step to build pipelines
* Run tests on root dir on pr
* Revert "Remove tests step to build pipelines"
This reverts commit 497bd86620.
* CHange name
* Remove working directory
* Add manual trigger
* Comment out cache
* Try to fix
* Revert "Try to fix"
This reverts commit 93d404b7d5.
* Disable failing test
* Updated release jobs to include fixes that Vince recommended from the mobile repo
* Updated the workflows with the fixes
that Vince recommended from mobile.
* Fix linter errors
Co-authored-by: Todd Martin <>
Co-authored-by: Michal Checinski <michal.checinski@outlook.com>
* Fix name
* Fix version reference
* Add release version bump to every job
* Fix release channel name
* Setup git branch
* Change branch name
* Fix name
* Fix download artifact
* Uncomment after testing
* Removed jslib
* Fix comments
* [ENG-71] updated release-web.yml to add deployment updates.
* [ENG-71] Added newline at the end of the file
* [ENG-71] Changed environment name.
* [ENG-71] Changed branch name again.
* Fixed error in brackets.
* Added release version
* [ENG-71] Updated all the other scripts to track deployments.
* [ENG-71] added channel
* [ENG-71] Added newline
* [ENG-71] Removed separate deployments for choco and snap
* [ENG-71] Removed separate deployments
* [ENG-71] Added newline
* [ENG-71] refined messages
* [ENG-71] Changed to separate out environment and description.
* [ENG-71] more changes to sepearate environment from description
* Job cleanup
* Job cleanup
* [ENG-71] More job cleanup
* [ENG-71] Updated to use commit instead of v2.
* [ENG-7] Fixed error referencing release-version
Co-authored-by: Todd Martin <>
* Check if any changes to commit
* Comment out checkout since there is a git switch
* Change base branch to DEVOPS-872_version_bump_not_fail
* Remove checkout version branch since git switch does that
* Revert "Change base branch to DEVOPS-872_version_bump_not_fail"
This reverts commit 8874702469.
* Release to QA branch
* Fix branch name
* Change release to automatically push new version
* Fix name of GH bot
* Add check if there is sth to commit
* Fix comments
* Fix comments #2
* Keep k8s deployment
* Fix newlines
* disabling snap for testing
* adding in environment logging
* adding in some Dry Run logic
* adding more protected dry run logic
* skipping long running steps for testing purposes
* testing out windowws-2022 runner
* Trying a different GitHub Action expression syntax
* manually disabling jobs because I don't have time to figure out GitHub's weird syntax
* updating nuget on windows-2019
* disabling the nuget version 6
* trying more GitHub Action syntax
* disabling choco for testing
* running test on Dry Run release
* changing the double quotes to single quotes inside the GitHub Action expression
* removing testing code
* updating more dry run logic
* reverting a branch name change
* Add option for beta channels in package.json for electron
* Change placement of build settings
* Add version
* Revert "Add version"
This reverts commit 2e09cc7d5e.
* Add electron version to package.json
* Move build to higher level
* Add upload beta autoupdate
* Revert version
* Updating the release workflow with the new channels
* restrict GitHub Release to only 'latest'
* fixing electron-builder configuration
* version bump test
* fixing words
* Commenting out the npm cache in the windows build job
* Adding back in the caching for the future
* Revert "version bump test"
This reverts commit b732d00841.
Co-authored-by: Joseph Flinn <joseph.s.flinn@gmail.com>
* Add trigger when browser build completes
* Download artifacts from workflow run trigger
* Fix branch quotes
* Remove workflow run event trigger
* Add trigger step for desktop build
* Add current branch name for test purposes
* Change the branch name
* Add token retrieval from KeyVault
* Revert "Change the branch name"
This reverts commit 696157dc81.
* Fix json in request
* Add printing variables
* Change way of creating json string
* Change name of kv secret
