Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix public OTA updates #166

Merged
merged 23 commits into from
Jun 12, 2024
Merged

Conversation

truemiller
Copy link
Collaborator

@truemiller truemiller commented Jun 3, 2024

  • public OTA is turned back on / fixed
  • added some eslint rules for electron directory

can test from rc34.2 -> rc34.3

@mohandast52 please see github publish config, as it's different from the previously discussed.

have removed channels attribute as this array applied to SnapStore channels, not GithubOptions

@truemiller truemiller changed the title (WIP) Fix public OTA Fix public OTA updates Jun 5, 2024
@oaksprout
Copy link
Collaborator

This worked for me, but two things I notice:

  • the notification says the app is restarting, but it doesn't. It just hides the app from the foreground – the app is still running, and visible in the tray
  • As I think more about it, I don't actually think this level of automation is desirable. Due to the sensitive nature of the app – financial – I think we should make users manually apply the updates

I suggest we keep this on a separate branch and merge with Mohan's work. Then assess from there. We need to have user safety as top of mind.

const publishOptions = {
const githubOptions = {
releaseType: 'draft',
publishAutoUpdate: true,
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@oaksprout commented that the user has to manually update it.

@truemiller truemiller changed the base branch from main to mohan/improve-update-experience June 11, 2024 12:02
@truemiller truemiller merged commit 17fa130 into mohan/improve-update-experience Jun 12, 2024
2 of 4 checks passed
@truemiller truemiller deleted the josh/public-ota branch July 26, 2024 17:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants