Improve the Experience of Unterstufen Pupil about kag-app HOT 13 CLOSED

?

from kag-app.

I still dont understand what you mean here

from kag-app.

Unterstufen people are not permitted to see the RPlan.
-> Hide the RPlan or show a message

from kag-app.

Problem: It takes long to load that information. It would be better to cache the user group so that it does not download this information every time.
@strifel Please implement the caching mechanic sometime (before the release).

from kag-app.

Will close this issue.

from kag-app.

I think the Method isn't even async. Let me check

(EDIT:

from kag-app.

But how can I access this without using an async function?
The only way I see is:
var groups = (await KAGApp.api.getAPIRequest(APIAction.GET_GROUPS)).getGroups();
I can't delete the await

from kag-app.

That's true but in this case it should take 0 seconds.
If this has to be used in a not async context, I have to fix this. Does it need to be?

from kag-app.

Yes, it needs to be synced. Right now it takes 1 second to complete and the build method gets called before initState() is completed. -> It shows an error while initState() isn't completed.

I will work on this issue when the method is sync.

from kag-app.

The method getAPIRequestSync does not work properly. It checks:

if (_isLogInNeeded(action) && action != APIAction.GET_USER_INFO && action != APIAction.GET_USERNAME)
So: For getGroups login is needed so the task can't be performed synchronously.

@strifel Please update the condition here. For now isLoginNeeded for getGroups is changed to false.

from kag-app.

The statement is completly fine.

from kag-app.

No. The reason requestSync was created was to be able to get the Groups synchronously. With your implementation/statement you can‘t access groups synchronously.

getGroups only works when the user is logged in -> either isLoginNeeded for getGroups needs to be false (which it doesn‘t) or the statement must be updated

from kag-app.

Oh. Maybe I had a little typo....

from kag-app.