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

Sorting states of a device #25

Closed
braindead1 opened this issue Jul 25, 2020 · 6 comments
Closed

Sorting states of a device #25

braindead1 opened this issue Jul 25, 2020 · 6 comments
Assignees
Labels
bug 🐛 Something isn't working

Comments

@braindead1
Copy link

New states of a device are added at the top. The implemented possibility to move states up and down is not working. I tried it directly after creating the new state and also after saving the configuration.

@Zefau Zefau added the bug 🐛 Something isn't working label Jul 25, 2020
@Zefau Zefau self-assigned this Jul 25, 2020
@Zefau
Copy link
Owner

Zefau commented Jul 25, 2020

This is very weird. Works perfectly fine for me. What browser are you using? Do you have any errors in the browser's console?

@braindead1
Copy link
Author

In Google Chrome and Microsoft Edge sorting does not work for me. Directly after opening Jarvis I get the following error in the console:

Function (blind) # undefined
overrideMethod @ react_devtools_backend.js:2273
value @ Log.js:37
(anonymous) @ blind.js:96
Promise.catch (async)
(anonymous) @ blind.js:96
is @ react-dom.production.min.js:211
bl @ react-dom.production.min.js:257
t.unstable_runWithPriority @ scheduler.production.min.js:19
Hi @ react-dom.production.min.js:122
ml @ react-dom.production.min.js:257
Js @ react-dom.production.min.js:239
(anonymous) @ react-dom.production.min.js:123
t.unstable_runWithPriority @ scheduler.production.min.js:19
Hi @ react-dom.production.min.js:122
qi @ react-dom.production.min.js:123
Yi @ react-dom.production.min.js:122
Gs @ react-dom.production.min.js:230
enqueueSetState @ react-dom.production.min.js:132
x.setState @ react.production.min.js:12
(anonymous) @ App.js:158
Promise.then (async)
(anonymous) @ App.js:156
Promise.then (async)
value @ App.js:151
os @ react-dom.production.min.js:212
hl @ react-dom.production.min.js:255
t.unstable_runWithPriority @ scheduler.production.min.js:19
Hi @ react-dom.production.min.js:122
fl @ react-dom.production.min.js:248
Js @ react-dom.production.min.js:239
Gs @ react-dom.production.min.js:230
Nl @ react-dom.production.min.js:281
(anonymous) @ react-dom.production.min.js:284
tl @ react-dom.production.min.js:240
Hl @ react-dom.production.min.js:284
t.render @ react-dom.production.min.js:290
637 @ index.js:8
i @ index.html:1
398 @ main.91c26d32.chunk.js:1
i @ index.html:1
t @ index.html:1
r @ index.html:1
(anonymous) @ main.91c26d32.chunk.js:1

There is no error in the console when I try to sort states.

@Zefau
Copy link
Owner

Zefau commented Jul 27, 2020

I created a separate issue for that log, because it has nothing to do with the sorting issue.

I was able to reproduce and fix the sorting issue. Will be available with the next release.

@Zefau Zefau added this to the ROADMAP v1 milestone Aug 14, 2020
@Zefau
Copy link
Owner

Zefau commented Aug 15, 2020

@braindead1 please verify if this got solved in v1.0.0-alpha.1, thanks

@braindead1
Copy link
Author

This issue is solved. I cannot reproduce the error in v1.0.0-alpha.3

@Zefau
Copy link
Owner

Zefau commented Sep 7, 2020

01d4615

@Zefau Zefau added release and removed verify-fix labels Sep 12, 2020
@Zefau Zefau removed this from the ROADMAP v1.0.0 milestone Sep 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants