[0.6.x] Link to release instead of updating #3619
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Mrcubix
changed the title
|
Now that i re-think about, it https://github.com/OpenTabletDriver/OpenTabletDriver/releases/latest exists |
Use latest instead of targetting specific tag
Mrcubix
force-pushed
the
Updater-Check-Link
branch
from
5c0ccae to
62cf2fc
Compare
gonX
reviewed
Dec 22, 2024
gonX
added
windows/wpf
gonX
reviewed
Dec 25, 2024
|
Since this will rely entirely on the user manually updating, it would be a good idea to list the folder path where OTD is. Maybe a readonly textbox so the user can copy paste it. |
|
Might as well have a button that open the directory then. |
Using diff --git a/OpenTabletDriver.UX/Windows/Updater/UpdaterWindow.cs b/OpenTabletDriver.UX/Windows/Updater/UpdaterWindow.cs
index 4f08a153..fd94acc0 100644
--- a/OpenTabletDriver.UX/Windows/Updater/UpdaterWindow.cs
+++ b/OpenTabletDriver.UX/Windows/Updater/UpdaterWindow.cs
@@ -2,6 +2,7 @@ using System;
using System.Threading.Tasks;
using Eto.Drawing;
using Eto.Forms;
+using OpenTabletDriver.Desktop;
using OpenTabletDriver.Desktop.Interop;
using OpenTabletDriver.UX.Controls;
using OpenTabletDriver.UX.Controls.Generic;
@@ -49,9 +50,22 @@ namespace OpenTabletDriver.UX.Windows.Updater
new Bitmap(App.Logo.WithSize(256, 256)),
"An update is available to install",
$"OpenTabletDriver v{updateAvailable.Version}",
- new Button(OpenRelease)
+ new PaddingSpacerItem(),
+ new StackLayout()
{
- Text = "Go to Release"
+ Orientation = Orientation.Horizontal,
+ Items =
+ {
+ new Button(OpenRelease)
+ {
+ Text = "Go to Release"
+ },
+ new Button(OpenDirectory)
+ {
+ Text = "Open Directory"
+ }
+ },
+ Spacing = 5
},
new PaddingSpacerItem(),
}
@@ -70,5 +84,8 @@ namespace OpenTabletDriver.UX.Windows.Updater
private void OpenRelease(object sender, EventArgs e)
=> DesktopInterop.Open(LATEST_RELEASE_URL);
+
+ private void OpenDirectory(object sender, EventArgs e)
+ => DesktopInterop.Open(AppContext.BaseDirectory);
}
} |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Open the Github release instead of using the faulty updater for now.
One thing to note is that since no updater object is provided for Linux, they aren't notified of new updates.
That is, until they manually check for one via their Package Manager.
Closes #3618