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

[MainUI] Inbox / add as thing: let the user decide the thing id #474

Closed
lolodomo opened this issue Oct 31, 2020 · 13 comments · Fixed by #775
Closed

[MainUI] Inbox / add as thing: let the user decide the thing id #474

lolodomo opened this issue Oct 31, 2020 · 13 comments · Fixed by #775
Labels
enhancement New feature or request main ui Main UI question Further information is requested

Comments

@lolodomo
Copy link
Contributor

When you add an inbox entry as thing, the pop-up propose to edit the thing label. It would be cool to have a way to edit the thing id (last part of the thing UID) too because it is something no more editable later.

@ghys
Copy link
Member

ghys commented Oct 31, 2020

Agreed, but it's not really a "webui" problem since the /rest/inbox/{thingUID}/approve API only lets you approve inbox entries where the thing ID has been already decided. I suggest moving it to core to propose a change to the API.

@lolodomo
Copy link
Contributor Author

Ok with your suggestion ;)

@lolodomo
Copy link
Contributor Author

@ghys : can you please reopen this issue.
I have implemented the feature on core side: openhab/openhab-core#1848

@ghys ghys reopened this Nov 25, 2020
@lolodomo
Copy link
Contributor Author

lolodomo commented Nov 26, 2020

Change in core is now merged.
@ghys : it would be super now if the MainUI could be enhanced to provide a new field in the popup to enter an optional thing id.

@ghys
Copy link
Member

ghys commented Nov 26, 2020

Okay, there's no dialog with 2 text fields available so there will be a need for a completely redesigned UI to approve.

@ghys
Copy link
Member

ghys commented Dec 13, 2020

Adding the question label until the problem with the API is fixed (see openhab/openhab-core#1848 (comment))

@ghys ghys added main ui Main UI question Further information is requested labels Dec 13, 2020
@lolodomo
Copy link
Contributor Author

@ghys : API is now fixed in core framework (PR merged).

ghys added a commit to ghys/openhab-webui that referenced this issue Jan 8, 2021
@ghys ghys closed this as completed in #775 Jan 8, 2021
ghys added a commit that referenced this issue Jan 8, 2021
@lolodomo
Copy link
Contributor Author

lolodomo commented Jan 10, 2021

@ghys : I don't understand how it was enhanced.
Here is the dialog box when I add a thing for the inbox:

image

As before, I have a control only on the label but not on the thing id.

OH3 snapshot 2131:

openhab> bundle:list -s | grep ui
 83 x Active x  80 x 3.10.400.v20190516-1504 x org.eclipse.equinox.common
160 x Active x  80 x 3.1.0.202101100957      x org.openhab.core.io.rest.ui
190 x Active x  80 x 3.1.0.202101100956      x org.openhab.core.ui
191 x Active x  80 x 3.1.0.202101100957      x org.openhab.core.ui.icon
193 x Active x  80 x 3.1.0.202101100100      x org.openhab.ui
241 x Active x  80 x 3.1.0.202101100057      x org.openhab.ui.basic
242 x Active x  80 x 3.1.0.202101100057      x org.openhab.ui.iconset.classic

@ghys
Copy link
Member

ghys commented Jan 11, 2021

When you click on a thing and get the menu you should have a new "Add as Thing (with custom ID)" option, if not that might be a cache problem, go to Help & About and select the purge caches option if you have it, otherwise Reload app.
image

ghys added a commit to ghys/openhab-webui that referenced this issue Jan 11, 2021
@lolodomo
Copy link
Contributor Author

Oh yes, I see the menu on my phone.
I will try to understand why the menu is not displayed on my PC while I cleared the
WEB browser cache.

@lolodomo
Copy link
Contributor Author

lolodomo commented Jan 11, 2021

@ghys : I know what is happening. Your menu is OK in the "Inbox" page. The problem is that this menu is not proposed in the "Add a new Thing xxxxxx" page.
There is a workaround, I just have to return to the inbox page before adding my thing. But this is something usual to search a thing and to add it directly from the "add a new thing" page.

@lolodomo
Copy link
Contributor Author

In fact, I am just discovering again my feature request #416 ;)

@ghys
Copy link
Member

ghys commented Jan 12, 2021

I didn't forget about #416, I regularly check issue lists, and it will probably come in conjunction with it when/if it gets implemented.
#775 was mostly about fixing #744 to include it as part of the 3.0.1 patch release, I just added it because it was safe to add and in the same file :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request main ui Main UI question Further information is requested
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants