Changes

add basic shared device login impl

add basic shared device login impl

added QR badge

added QR badge test

Add build environment variable for default app list
Remove non-functional try button
Learning unit screen: remove non-functional sorter
Add progress bar to learning unit viewer.

Bump version for internal testing update.

Update db names and pref names to avoid update conflict.

Use default app list set by environment variable when provided.

Set versions for public app store.

Remove assertVisible try it button which is often unused (and now will not be shown).

Update RespectCredential sealed class to add QR Code Badge credential type.

add shared device login impl

remove shared device/settings screen changes from assign badge to a student flow

remove shared device/settings screen changes from assign badge to a student flow

add refactor

add qr login flow

add refactor

add refactor

add refactor

test added

refactor

refactor

refactor

add test tags

test added

updated test

add validation

updated test

updated test

fix test

add validation

updated test flow

updated test

update create accound screen

fix test

fix test

fix test

fix test

fix test

updated test flow

fix  popup menu

fix revoke qr

code refactor

updated test as per comments

updated test as per comments

updated test- added assignment flow

updated test- added assignment flow

fix comments

fix cconflict

updated test- added assignment flow

updated test- added assignment flow

fix qr code delete function

fix build failure

refactor

Remove gap on top of Scan QR code screen that should not have been there.

updated test-resolved merge conflicts

Minor tidyups:

a) Rename QR code screen and viewmodel package to scanqrcode (not shareddevicelogin) because the Scan QR code function will be used for more than just badge login
b) Minor formatting fixes for AppBar (avoid line overrun)

ScanQRCodeViewModel: should accept account manager as a normal dependency
Remove duplicate error messages from uistate. Should have only one error message.
Remove UiStates isloading - this should use launchWithLoadingIndicator and appstate's loading state (which is connected to the appbar and will automatically show a progress indicator accordingly).

fix test

add invalid qr code ui part

remove qrCodeBadgeError from manageaccountviewmodel

updated test - error

fix landscape mode

Change AppActionButton contentDescription to use UiText instead of string.

Fix ManageAccountViewModel : remove race condition created by calling .launch twice for a single operation.

Refactor for consistent naming on PersonQrBadge entity.

Previously some parts were called PersonBadge, some PersonQr. Needs to be consistent.

change qr code library

Tidyup ManageAccountScreen: if password not set, still use the same ListItem.
Remove includeDeleted from PersonQrBadgeDataSource.GetListParams - this is already available in common
Change PersonQrBadgeEntity Url field to use Url type.
Implement permission enforcement in PersonQrBadgeDataSourceDb

Further tidyup:
Ensure event functions are named onEventName eg onClickButton
Move Badge Info box into its own function

Tidyup ScanQRCodeViewModel:

a) Remove unnecessary try-catch statements: multiple try catch statements were all the same: if an exception is caught, set uiState error text to .uiTextOrGeneric
b) Remove unnecessary extra launch statements.

refactor

updated test - More Options

Remove PersonQrBadge existsByUrl: looking for a badge by URL needs to be done by using the list function.

Further qr badge datasource tidyup:
1) Remove unnecessary extension function (even though it was following the same pattern we had used for password)
2) Remove unused DAO query function
3) Add index for QR code url

Minor tidyup on SetUsernameAndPassword:
1) Remove unused ui state property isQrAlreadyAssigned
2) Rename uistate property: not isStudent, should be showQrBadgeInfoBox

AuthenticateQrBadgeUseCaseDbImpl: remove useless if statement that could never be true

qrCodeEntity.pqrQrCodeUrl != credential.qrCodeUrl would never be - it was found by searching the database by Url.

Remove hardcoded english from ui text strings

add automatic username suggestion

fix revoke issue

updated test - automated username field

updated test - automated username field

Update datalayer README to add conceptual notes on data deletion.

updated eraseText

Fix UrlExt.schoolUrlOrNull function.

remove other custom logic for schoolurl

updated test - automated username field

Fix PersonQrBadgeEntityDao LIST_SQL query: the WHERE clause was not handling guidNum = 0 correctly.

updated eraseText

Add normalization for school endpoint URL.

add qr code image

code refactor

updated test - automated username field

code refactor

updated test - automated username flow

Rename ManageAccount route parameters and add property KDocs.

remove authenticateqrbadgeusecase from client di

Update CreateAccountSetUserNameViewModel:

1) Simplify use of suggestedUsername: ensure that the use case is only invoked once. Because it can connect to the server, put the personDataSource collection into collectLatest so it can be cancelled.

2) Use existing validatepasswordusecase.

remove validatepassword fun

Updated password error

Updated password error

code refactor

Fix OtherOptionsViewModel to use normalized endpoint URL string when URL entered manually.

code refactor

Terminology change: class edit screen to show 'class name', assignment edit screen to show 'assignment name'.

Terminology change: class edit screen to show 'class name', assignment edit screen to show 'assignment name'.

Updated Name* to Class name* /Assignment name*

Tidyup:
a) Make PersonQrBadge qrUrl nullable: if no QR badge is assigned, we don't want to delete the PersonQrBadge itself. The Url should be set as null.
b) Tidyup ManageAccountViewModel

Temporarily disable checking invalid QR code scanned: this check should be in manage account screen.
Fix refactored ScanQRCodeViewModel (incorrectly had ! before sendResultIfExpected)

Add try/catch to avoid CreateAccountSetUserNameViewModel crashing if offline.