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

feat: add version incompatability check & modal #103

Draft
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

CrabNejonas
Copy link
Contributor

This checks the instrumentations version in the frontend to make sure its version isn't too outdated (the compat behaviour here is very rudimentary) and pop up a modal instructing the user to update the rust crate.

resolves DR-623

atila-crabnebula and others added 7 commits November 22, 2023 14:22
This checks the instrumentations version in the frontend to make sure its version isn't too outdated (the compat behaviour here is very rudimentary) and pop up a modal instructing the user to update the rust crate.

resolves DR-623
Copy link

linear bot commented Nov 22, 2023

DR-623 Devtools Version Compatibility

We are going to provide a way for people to be informed in the devtools console about the results of Tauri Info, which informs the user if their dependencies are up to date. But what if their tracing-plugin isn't up to date? When there is a compatibility mismatch, we need to have a strategy to:

  1. Inform
  2. Coach on how (what) to update
  3. Wait for them to fix

Copy link
Contributor

@tejas-crabnebula tejas-crabnebula left a comment

Choose a reason for hiding this comment

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

TS looks good to me.

@CrabNejonas CrabNejonas marked this pull request as draft November 29, 2023 10:13
@CrabNejonas
Copy link
Contributor Author

CrabNejonas commented Nov 29, 2023

Lets move this to post MVP and discuss how we properly set incompatible versions

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.

3 participants