WEKIT-ECS / MIRAGE-XR

MirageXR is a reference implementation of an XR training system. MirageXR enables experts and learners to share experience via XR and wearables using ghost tracks, realtime feedback, and anchored instruction.
Other
27 stars 4 forks source link

Implement the settings menu #92

Closed wekitecs closed 2 years ago

wekitecs commented 3 years ago

In GitLab by @BenediktHensen on Sep 20, 2020, 11:06

See #58 for details on the design.

The user data should be accessible by other scripts in the application. The fields that contain URLs should change the URLs in the corresponding scripts.

Bonus: Make the set values persistent on one device, e.g. using PlayerPrefs

Related to: https://platform.xr4all.eu/wekit-ecs/mirage-xr/issues/275 and https://platform.xr4all.eu/wekit-ecs/mirage-xr/issues/222 and https://platform.xr4all.eu/wekit-ecs/mirage-xr/issues/179 (display version number on config panel) and https://platform.xr4all.eu/wekit-ecs/mirage-xr/issues/275 (config file)

wekitecs commented 3 years ago

In GitLab by @BenediktHensen on Sep 20, 2020, 23:44

This menu has a lock at the top right. To realize its functionality, incorporate the general solution that was/will be created for #103.

wekitecs commented 3 years ago

In GitLab by @BenediktHensen on Sep 27, 2020, 11:58

created branch features/92-implement-the-settings-menu to address this issue

wekitecs commented 3 years ago

In GitLab by @fwild on Oct 4, 2020, 14:58

BTW: the line where the mbox username for the xAPI is stored is lines 13 and 31 in: https://platform.xr4all.eu/wekit-ecs/mirage-xr/blob/features/redesign/Assets/MirageXR/Common/ExperienceAPI/ExperienceAPIService.cs

wekitecs commented 3 years ago

In GitLab by @wild on Feb 16, 2021, 17:56

Related to https://platform.xr4all.eu/wekit-ecs/mirage-xr/issues/275

wekitecs commented 3 years ago

In GitLab by @william.guest on Mar 5, 2021, 17:45

since this is 'priority low', i would suggest moving it to sprint 6.

wekitecs commented 3 years ago

In GitLab by @wild on Mar 7, 2021, 21:19

moved.

wekitecs commented 3 years ago

In GitLab by @a85jafari on Mar 8, 2021, 01:02

After Moodle integration the user information should be edited on Moodle. Already we have a setting view for moodle and a ticket #275 will save all settings. I think we should close this ticket

wekitecs commented 3 years ago

In GitLab by @wild on Mar 8, 2021, 10:36

There are additional URLs and options we need to configure, not just Moodle: xAPI URL, kid-friendly UI yes/no, IBM Watson credentials (in the near future), ... so we still need this...

wekitecs commented 3 years ago

In GitLab by @a85jafari on Mar 17, 2021, 15:52

@wild currently we do not have any of those feature you named, and I do not think we will have in sprint 6 neither!

wekitecs commented 3 years ago

In GitLab by @wild on Mar 18, 2021, 11:51

We do have the mbox name and LRS url - in the Experience Manager. I have just checked on https://lrs.tech4comp.dbis.rwth-aachen.de/login (use the credentials in the code) - but it seems not to log any data at the moment. Strange. I will add a separate ticket for that - but having the URL configurable is still necessary:

line 13 of ExperienceAPIService.cs: url: https://lrs.tech4comp.dbis.rwth-aachen.de/data/xAPI user: mirageXRdemo@gmail.com pwd: wekit-ecs.com

line 31 of the ExperienceAPIService.cs mbox: we should compose the mbox name automatically from the username we use for Moodle in the format of: wild@arete.ucd.ie - this is used for identifying users only, and we can interface later with a dashboard widget to display the analytics in the app as well, not just in the LRS

wekitecs commented 3 years ago

In GitLab by @BenediktHensen on Mar 18, 2021, 16:09

This issue was essentially finished two sprints ago. However, we then introduced the Platform Manager which made it necessary to also implement the mobile version.

Since then, the requirements have changed a lot, e.g. with the integration of the Moodle login and new settings like the Kids UI.

Hence, the settings menu needs a revision to update the solution.

wekitecs commented 3 years ago

In GitLab by @BenediktHensen on Mar 18, 2021, 16:09

unassigned @BenediktHensen

wekitecs commented 2 years ago

In GitLab by @wild on Sep 10, 2021, 13:08

We have changed plans slightly over time and this ticket is not needed anymore (largely implemented / different solution via config files).