Open denis99999 opened 5 years ago
Issue-Label Bot is automatically applying the label feature_request
to this issue, with a confidence of 0.89. Please mark this comment with :thumbsup: or :thumbsdown: to give our bot feedback!
Links: app homepage, dashboard and code for this bot.
I’ve only been able to briefly scan through the PDF - which is great, by the way - so far, but this seems like an excellent addition. Unfortunately I myself have very limited time to test it out, so it would be very helpful if anyone interested in device booking could provide their feedback. I know there’s a bunch of you out there :)
One thing I picked up was that the PDF mentioned a possible need to nuke RethinkDB once. Is that a hard requirement?
Yes, the Rethinkdb database has to be rebuilt from scratch otherwise it will does not work! But it is easy to save in a secure way the existing DB before to make the test (ex: # mv /opt/rethinkdb /opt/rethinkdb.org)
@denis99999 I've went through the design doc, which shows a comprehensive solution for user/group/device management. I will try to test the feature and feedback.
Hi @thinkhy, Thank you very much for your contribution, don't hesitate to contact me for any question!
Hello @denis99999, can you please give me some advices how to install your feature?
Hi @pawbednarz,
You have simply to clone https://github.com/denis99999/stf.git
, then switching to the group-feature
branch and follow the STF installation instructions.
Once it is done, you have to read the PDF documentation joined in this PR to well understand how device reservation works!
And finally enjoy :-)
Don't hesitate to contact for any other question!
Thanks @denis99999, i just forgot to switch the branch. Very good job with this improvement! Greetings!
Great work @denis99999
Do you know if there is any ability of using stf with admin rights. For example, Scenario-
Thanks Shiva
Hi @skumar1122 ,
Yes of course, I developed the booking/partitioning system for this kind of purpose, using it you have just to reserve for you the considered few devices for the time you need! For detailed information, take a look to the PDF documentation I joined to this PR. Regards. Denis.
Hey @denis99999
Thanks for the reply. I am checking above document and I couldn't see STF start instructions
with your feature branch. Could you point me please ?
It runs(stf local
) same like before and I am seeing new changes like you mentioned in your document.
Thanks Shiv
See sections 9 & 10, setting correctly NTP server and rethink database, all is explained in the PDF with a complete example, you have just to read ! Regards. Denis.
Sorry,I did see this section and I logged in with admin details and I am still seeing old GUI. Do I need to start STF in different way and I am using Mac OS.
9 System configuration Obviously, it is required to get time synchronization with accurate time servers (e.g. using NTP on Linux) on the STF machines, in order the booking & partitioning systems work fine.
10 Database migration The Group feature introduced into STF require some built-in objects in the database to work fine:
It seems you did not switch to the group-feature branch, isn't it ? remind also you have to rebuild the database (i.e. remove then create)
Hi,
I've played a while with your fork and I'm impressed :) I still faced some issues:
First: It looks a bit like DB schema is not compatible and causing following kind of errors:
```
2019-12-10T05:36:42.351Z ERR/processor 21538 [proc002] Unable to save the initial state of Device "H8WDU15B02000760" TypeError: Cannot read property 'email' of undefined
at /home/tester/stf/lib/units/processor/index.js:106:40
at runCallback (timers.js:794:20)
at tryOnImmediate (timers.js:752:5)
at processImmediate [as _immediateCallback] (timers.js:729:5)
From previous event:
at Router.
I'm not sure if following errors related to same:
```
2019-12-10T05:34:33.956Z INF/device:plugins:mute 21406 [FA542YJ13460] Will not mute master volume
ReqlDriverCompileError: Cannot wrap undefined with r.expr().
at ReqlDriverCompileError.ReqlError [as constructor] (/home/tester/stf/node_modules/rethinkdb/errors.js:23:13)
at ReqlDriverCompileError.ReqlCompileError [as constructor] (/home/tester/stf/node_modules/rethinkdb/errors.js:35:51)
at new ReqlDriverCompileError (/home/tester/stf/node_modules/rethinkdb/errors.js:46:57)
at Function.
But propose to check those out.
NOTE: After I swipe whole DB and restart stf again all above exceptions go away and server started smoothly. Also my android 10 device appears, great job! I quickly played with it - installed apk and started it - so far everything works as expected from android 10 point of view !
One minor UX issues I faced:
renaming group could go a bit smoother - approval should close edit mode, see screen capture below:
My environment:
Hi @jupe ,
Yes, as explained in the PDF documentation about Group feature I joined, STF database has to be rebuilt, see also the discussion #1057 about that.
Concerning the UI, thank you for your proposal but the logic is to click on the "blue" buttons to open/close the corresponding windows, while a validation button as for "Name" or "Schedule" has the semantic to commit the current edition, but not to stop the edition, and is active only in case of change, so I feel it would be not consistent to use such button to shortcut the closing of the window.
Hi,
The related pull request #1056 named "Group feature" is made of:
This Group feature has been added into STF taking care to maintain the backward compatibility with third party client software using STF APIs.
Moreover, although several screens and tabs have been added to the native STF UI to serve the considered new feature, the backward compatibility is also maintained at UI level so that users can continue to use STF as before if they have no need to use the booking & partitioning systems.
Nevertheless, due to the adding of new built-in objects into the database impacting existing tables, it is necessary to rebuild the database by erasing it first (i.e. only access tokens may be easily saved).
Here is a full documentation about this new feature including also some implementation details: GroupFeature-1.0-STF-OrangeSA.pdf
Don't hesitate to test it from here: https://github.com/denis99999/stf/tree/group-feature