teammates
teammates copied to clipboard
Show loading message below response loading spinner
From PR #10960
Description of feature/enhancement
If a course has many students (or in general many responses per question), add extra loading text below the spinner to inform the user that the students are being loaded (after perhaps 5s).
Justification
From @damithc
Provided the load operation will not time out, perhaps simply show a message below the spinner cautioning the user that this particular question will take a long time to load?
This would give user more assurance that there is progress in the loading.
Existing similar features and their shortcomings
Experimented with using forkJoin
in the frontend to fetch questions in parallel in #10960: the parallel calls create too much instantaneous load at the backend and causes new instances to be spun up. The cost here is not justified.
I will take this issue
Hi. Is this issue resolved? Or anyone working still on it?
@anshuman-dev Priority will be given to the previous commenter as they have a PR up. You can try to find another good first issue
to work on
Is this issue resolved?
Is the issue resolved? I would like to take this up.
@shivamsharma01 The issue is still open, you can work on it
@shivamsharma01 The issue is still open, you can work on it
I am not able to setup the project properly and getting some errors. Can anyone please assist?
While trying to run gradlew.bat serverRun getting the below error 2022-01-18 13:55:13.363:WARN:oejw.WebInfConfiguration:main: Web application not found D:\Open%20Source\teammates\build\exploded-teammates
Also, on the UI when i try to login as student or instructor i am getting the below error on console Access to XMLHttpRequest at 'http://localhost:8080/webapi/auth?frontendUrl=http://localhost:4200' from origin 'http://localhost:4200' has been blocked by CORS policy: Response to preflight request doesn't pass access control check: No 'Access-Control-Allow-Origin' header is present on the requested resource.
@shivamsharma01 Hi, for the first error it may be because your folder Open Source
contains a space in the folder name? Maybe can try changing the folder name to OpenSource
or something that do not contain a space.
Is this issue still open? I would like to work on it.
Yes, probably. I was able to setup the workspace with the help of @hhdqirui 's suggestions but now I am not able to login to the application. Don't know how to create a user.
Can I also attempt working on this issue if it's still not resolved?
@rom-me Hi, please look for another issue to work on as there are previous users who wish to tackle this
@shivamsharma01 you can refer to https://github.com/TEAMMATES/teammates/blob/master/docs/development.md to see how to create users
In my understanding, there is a switch between the loading-spinner component (via loading-spinner directive) and the templateref using ngIf directive. There is no direct way to show the message while the loader is being displayed through the parent component. I have added the logic to display 'Loading Records...' after 5 seconds but this is now not limited to records being loaded in the feedback page but every time the loader component is visible for more than 5 seconds on any screen. Kindly let me know if instead this should be limited to only a specific case and any suggestions to handle it.
Can I take this issue?
@hlee18lee46 Go ahead!
Can I work on this issue, or is it already assigned to someone?
@PrakritiShetty Someone else has taken it. Please find other issues to work on instead!
Assign this issue to me
@sumit970 Hey! Are you still working on this? If not, I'd like to work on it
Hi, I would like to take this issue up
@sumit970 Hello, are you still working on this issue? If not, we can have others to take this up.
@MMilou24 Hey there! Let's give @sumit970 some time to respond. If not, you may look at other issues here for first time contributors.
hey you can work on it.
@sumit970 Thanks for getting back! Much appreciated.
@MMilou24 Feel free to work on this issue.
@MMilou24 and I are working on this together for a university course, therefore I have created the pull request.