HGustavs / LenaSYS

Code Viewer and Course Organization System used in some courses at University of Skövde
70 stars 31 forks source link

Re-engineer microservice getUid #17601

Open a23edvto opened 1 month ago

a23edvto commented 1 month ago

Follow the "coding standard for removing includes" for this file in folder /sharedMicroservices. The coding standard can be found in a couple of files in /microservices/Microservices Documentation. Now the functions in DuggaSys/microservices/curlService.php can be used.

Note: This one takes time

a23louma commented 4 weeks ago

Before I can start with the re-engineering of getUid_ms.php I need to go through all inverse dependency files, there are right now 50 files listed in the documentation and some are still missing. I will create a sub-issue for adding these to the inverse dependency documentation.

a23louma commented 3 weeks ago

After going through the inverse dependencies there are 60 microservices using getUid_ms.php that will be changed in this issue.

a23louma commented 2 weeks ago

I've been working on this issue for several days but have not being able to make it work after changing all 60 files with the curlService function callMicroserviceGET. I'm getting many errors in LenaSYS and have not been able to find what the problem is. I think someone else should try with this one as well.

a23louma commented 2 weeks ago

Probably this issue has to be solved by changing to use curlService POST instead, as GET might not work.