lodestone icon indicating copy to clipboard operation
lodestone copied to clipboard

290 desktop client should not force user to setup a local core

Open jhuang38 opened this issue 2 years ago • 8 comments

Description

Added way for users to skip local core setup in initial page load in Tauri: image

Added extra button in core select page in Tauri for local setup later on - this works by selecting the first option that shows up in Tauri, which might not work in all cases. image

Type of change

  • [ ] Bug fix (non-breaking change which fixes an issue)
  • [x] New feature (non-breaking change which adds functionality)
  • [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • [ ] This change requires a documentation update

Note: make sure your files are formatted with rust-analyzer

jhuang38 avatar Aug 01 '23 05:08 jhuang38

Deploy Preview for lodestone-dashboard ready!

Name Link
Latest commit bb7aad329478955f869544f67cfe236d0d815931
Latest deploy log https://app.netlify.com/sites/lodestone-dashboard/deploys/64e94b41880f4200088395b5
Deploy Preview https://deploy-preview-324--lodestone-dashboard.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

netlify[bot] avatar Aug 01 '23 05:08 netlify[bot]

Deploy Preview for lodestone-storybook ready!

Name Link
Latest commit bb7aad329478955f869544f67cfe236d0d815931
Latest deploy log https://app.netlify.com/sites/lodestone-storybook/deploys/64e94b41b3584c000740069c
Deploy Preview https://deploy-preview-324--lodestone-storybook.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

netlify[bot] avatar Aug 01 '23 05:08 netlify[bot]

We need to disable this popup in this usecase (tauri & localcore not setup & on dashboard page looking at external cores)

image

Ynng avatar Aug 10 '23 00:08 Ynng

This page might benefit from a "back" button

image

Ynng avatar Aug 10 '23 00:08 Ynng

This button still says "setup" even if the local core is already setup.

We should remove this button, or change the text to be more general like "use local core".

image

Ynng avatar Aug 10 '23 01:08 Ynng

This page might benefit from a "back" button

image

This page might benefit from a "back" button

image

which page should we go back to? in this issue there are 2 ways of going to that page, from this one: image

and this one: image

jhuang38 avatar Aug 15 '23 19:08 jhuang38

We need to disable this popup in this usecase (tauri & localcore not setup & on dashboard page looking at external cores)

image

could you clarify on the exact steps you did to get this? i can't seem to get to this popup

jhuang38 avatar Aug 15 '23 19:08 jhuang38

Regarding back button, I think the default browser "navigate backward" behavior might be enough. For the "new local core detected" prompt, reproduce it like this:

  1. setup lodestone core on a separate machine
  2. delete your local lodestone core
  3. open lodestone desktop, click connect to existing core
  4. enter details and get to the dashboard
  5. it should show up

Ynng avatar Aug 19 '23 21:08 Ynng