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

Update dependency tree #28

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

Conversation

matthiasfabry
Copy link

@matthiasfabry matthiasfabry commented Sep 24, 2024

Major update of the UI source code to be compatible with the latest React, React-Router and Electron.
I think all functionality should work, but since I have little experience with how the UI should be used, I might be wrong.
Also, packaging the electron app on my machine makes it display nothing (as in, electron launches an empty blue window). I don't know what's up with that, but at least the development version works (npm run electron).

Please test thoroughly before merging.

Closes #27

@matthiasfabry
Copy link
Author

Oh, and as d98f822 says, only http works for now. We might want to carry around a protocol variable later to easily switch over to https.

@kecnry
Copy link
Member

kecnry commented Sep 25, 2024

Wow, this was a huge effort, thanks! I'll try to put aside some time to do a full review and some testing in the next few weeks

@kecnry
Copy link
Member

kecnry commented Oct 23, 2024

I am seeing everything stacked vertically instead of in responsive columns, any idea why that might be?

@aprsa
Copy link

aprsa commented Nov 15, 2024

I can't even get the ui to connect to the local server, so it'll take me some time to troubleshoot.

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.

Dependency tree outdated
3 participants