llaske / sugarizer-server

Sugarizer Server allow deployment of Sugarizer on a local server, for example on a school server.
Apache License 2.0
43 stars 94 forks source link

recent assignment deliveries #403

Open parteekcoder opened 1 year ago

parteekcoder commented 1 year ago

Hi @NikhilM98 @ricknjacky ,

I am glad to work on this issue, I would like to mention what I have completed till now

Now will fix the tests which got affected by the changes in the endpoints , so for now github test may get failed

I would like to hear any suggestions from you on this PR and I will work upon it

parteekcoder commented 1 year ago

I created some assignments using the teacher account and submitted that from the student account Here is the recent assignment deliveries widget which contains the recently submitted assignments image

parteekcoder commented 1 year ago

image

these boxes will automatically adjust according to the number of rows and align themselves in new rows when they contain more number of row

parteekcoder commented 1 year ago

@NikhilM98 @ricknjacky

All these tests are passing because on checking the coverage report I analyzed that there is no test exists for the findAllDeliveries controller Here is the coverage report for findAllDeliveries image

image

I will increase the coverage for this by writing tests for this in different scenarios, I am improving coverage at #402

parteekcoder commented 1 year ago

I wrote 2 new tests for this endpoint And increased the coverage image

image

parteekcoder commented 1 year ago

Hi @NikhilM98 @ricknjacky ,

Now you can review this PR. I am open to new suggestions and will work on them to improve this PR