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

Tauri v1 -> v2 #4789

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Tauri v1 -> v2 #4789

wants to merge 5 commits into from

Conversation

mtsgrd
Copy link
Contributor

@mtsgrd mtsgrd commented Aug 29, 2024

  • rewrite of app updater because of new api
  • webkit2gtk 4.0 -> 4.1 upgrade, will now work without appimage on Ubuntu 24.04
  • webdriver actions API now working

Copy link

vercel bot commented Aug 29, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
gitbutler-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 24, 2024 5:16pm

@honnip
Copy link

honnip commented Aug 31, 2024

Part of #4694

@mtsgrd mtsgrd changed the title Tauri v1 -> v2 at the point where it compiles and loads Tauri v1 -> v2 Aug 31, 2024
@mtsgrd mtsgrd marked this pull request as ready for review August 31, 2024 22:16
- different dependencies -> different cache
@@ -27,7 +27,7 @@
loading = true;
try {
// TODO: Refactor temporary solution to forcing Windows to use system executable
if ($platformName === 'win32') {
if (platformName === 'windows') {
Copy link
Member

Choose a reason for hiding this comment

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

have we tested that this works?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

They are enumerated so typescript complains if you are using an incorrect value.

* The application metrics can be enabled or disabled by setting the value of the store to true or false.
* @returns A writable store with the appMetricsEnabled config.
*/
readonly appMetricsEnabled = this.persisted(true, 'appMetricsEnabled');
Copy link
Member

Choose a reason for hiding this comment

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

these still work correctly, right?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes, but let me also test this manually before landing these changes.

[ -z "${APPLE_ID-}" ] && error "$APPLE_ID is not set"
[ -z "${APPLE_TEAM_ID-}" ] && error "$APPLE_TEAM_ID is not set"
[ -z "${APPLE_PASSWORD-}" ] && error "$APPLE_PASSWORD is not set"
export APPLE_CERTIFICATE="$APPLE_CERTIFICATE"
export APPLE_CERTIFICATE_PASSWORD="$APPLE_CERTIFICATE_PASSWORD"
export APPLE_SIGNING_IDENTITY="$APPLE_SIGNING_IDENTITY"
Copy link
Member

Choose a reason for hiding this comment

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

we dont need this anymore?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I remember nightly failed to build unless I removed it, but I think we should investigate further.

https://github.com/tauri-apps/tauri/blob/a8105eccb2122f003b4a36bbfb747998f53c5f10/crates/tauri-cli/src/interface/rust.rs#L1299-L1306

Refactor appSettings to accommodate new Tauri v2 API

- creates AppSettings class and injects it where needed
- avoids `window` undeclared variable during vite build process
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
@gitbutler/desktop rust Pull requests that update Rust code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants