ILIAS icon indicating copy to clipboard operation
ILIAS copied to clipboard

T&A: Feature: New option for excel export of ILIAS test statistics

Open fhelfer opened this issue 2 years ago • 2 comments

Feature Wiki: https://docu.ilias.de/goto_docu_wiki_wpage_7405_1357.html

fhelfer avatar Oct 10 '22 07:10 fhelfer

As the client, did Alina check it out and give her OK?

dsstrassner avatar Oct 10 '22 09:10 dsstrassner

I requested an approval instance.

mbecker-databay avatar Oct 11 '22 11:10 mbecker-databay

Hi @fhelfer, since the deadline for ILIAS 9 is moved to 2023, I see the chance for refactoring this complex methods to improve the T&A code wholesome. @klees will review this PR and suggest some improvements due to his engagement in the TA TechSquad.

@alinaseibt and I will test the whole implementation afterwards at the approval instance and if everything works like a charm, write also the test cases for TestRail and add the related information to the FR Wiki page.

dsstrassner avatar Oct 19 '22 11:10 dsstrassner

I am good with this one! Please advise once this one can be merged.

mbecker-databay avatar Dec 12 '22 09:12 mbecker-databay

@dsstrassner If you can give us a poke about this, we can merge it while it still smells good.

mbecker-databay avatar Dec 12 '22 11:12 mbecker-databay

Update: @fhelfer will provide a second instance. I will check the implementation there again and comment afterward.

dsstrassner avatar Jan 09 '23 10:01 dsstrassner

I tested the feature along the Feature Request and everything works now as expected on the approval instance. I am ok with merging it @mbecker-databay ;)

Edit: 12:56h: Alina also tested it and only gets my test runs and I get now an error: "Sorry, an error occured. A logfile has been created which can be identified via the code "cjln6_6516"." Maybe we have to wait a little longer ...

dsstrassner avatar Feb 10 '23 11:02 dsstrassner

Edit: 12:56h: Alina also tested it and only gets my test runs and I get now an error: "Sorry, an error occured. A logfile has been created which can be identified via the code "cjln6_6516"." Maybe we have to wait a little longer ...

Fixed It was an Ilias Bug that had been fixed for 8. So rebasing did the trick

fhelfer avatar Feb 10 '23 13:02 fhelfer

Now everything works fine. Thanks a lot @fhelfer

dsstrassner avatar Feb 16 '23 07:02 dsstrassner