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

[desktop] Registry support #459

Merged
merged 22 commits into from
Nov 9, 2023
Merged

Conversation

Janaka-Steph
Copy link
Contributor

@Janaka-Steph Janaka-Steph commented Nov 1, 2023

Fix fetching current registry.
Add additional registries.
It closes #454

@Janaka-Steph Janaka-Steph marked this pull request as ready for review November 2, 2023 11:09
src-tauri/src/controller_binaries.rs Outdated Show resolved Hide resolved
src-tauri/src/controller_binaries.rs Outdated Show resolved Hide resolved
src-tauri/src/controller_binaries.rs Outdated Show resolved Hide resolved
src-tauri/src/controller_binaries.rs Outdated Show resolved Hide resolved
src-tauri/src/controller_binaries.rs Outdated Show resolved Hide resolved
src-tauri/src/controller_binaries.rs Outdated Show resolved Hide resolved
src-tauri/src/controller_binaries.rs Outdated Show resolved Hide resolved
src-tauri/src/controller_binaries.rs Outdated Show resolved Hide resolved
src-tauri/src/main.rs Outdated Show resolved Hide resolved
src-tauri/src/main.rs Outdated Show resolved Hide resolved
@tiero
Copy link
Contributor

tiero commented Nov 2, 2023

If I add a registry, then navigating to main page does not reflects the update unless I reload (ie/ close and reopen the app) is this a wanted behavior?

Also at first load I see "No service found" only way is go Settings, remove, add again and close/reopen the app

@tiero
Copy link
Contributor

tiero commented Nov 2, 2023

Please add a button "Reset" to use the default registry URL in case the user removes all of them

src-tauri/src/utils.rs Outdated Show resolved Hide resolved
Copy link
Contributor

@swarnimarun swarnimarun left a comment

Choose a reason for hiding this comment

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

lgtm, just a couple nits for some performance.

tiero and others added 2 commits November 7, 2023 11:54
@tiero
Copy link
Contributor

tiero commented Nov 7, 2023

cargo check fails @swarnimarun

@tiero
Copy link
Contributor

tiero commented Nov 7, 2023

@swarnimarun Please can you properly update the code using your siggestions?

It does not compile otherwise with a error[E0308]: mismatched types

tiero and others added 11 commits November 7, 2023 17:17
* use v1 branch of prem registry

* Fix chat history (prem-research#439)

* Available tag (prem-research#433)

* Available tag

* filter available + rm appId

* rm unused var

* gha: workflow_dispatch with manual version and branch name

* gha: debug

* gha: use version as number

* v0.1.3

* remove mp3 (prem-research#432)

* Update to TanStack Query v5 (prem-research#443)

* fix: binary_url precedence logic (prem-research#450)

* fix: binary_url precedence logic

* fix: lint

* Remove isFetching (prem-research#460)

* fix: match premd routes with slash in docker controller (prem-research#455)

* fix: gracefully handle process kill (prem-research#442)

* GHA: split Tauri and Docker in two workflows (prem-research#462)

* chore: add docker-compose to test app, daemon and gateway

* docker-compose.gateway point to stable premd

* v0.1.4

* fix: buggy state fetch (prem-research#471)

* v0.1.5

---------

Co-authored-by: Janaka-Steph <[email protected]>
Co-authored-by: Biswaroop <[email protected]>
Co-authored-by: Swarnim Arun <[email protected]>
* Skip healthcheck

* Skip healthcheck
* Petals banner

* Petals banner

* fiw warning
Copy link
Contributor

@swarnimarun swarnimarun left a comment

Choose a reason for hiding this comment

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

tested the new suggestions locally, also note, currently the project won't build on windows, should I fix it ??

src-tauri/src/controller_binaries.rs Outdated Show resolved Hide resolved
src-tauri/src/utils.rs Outdated Show resolved Hide resolved
@tiero
Copy link
Contributor

tiero commented Nov 8, 2023

currently the project won't build on windows, should I fix it ??

Project works only for Mac M1, and we dont have binaries for Linux/Windows even

@swarnimarun
Copy link
Contributor

@tiero I think we can merge now, what do you think?

@tiero tiero merged commit 8a7cd33 into prem-research:main Nov 9, 2023
1 check passed
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.

[Desktop] Update Registry URL
3 participants