Recursively traverse nominated GitHub directories for reports - support / or 'empty'

Description

As noted in UILDP-17, and the templated queries, i.e. reports, that are available to the LDP app are specified by a set of directories within GitHub repositories. This set is maintained as part of the app's settings (UILDP-97).

solves inclusion of reports from the top level using . as the path.

Scope: This ticket is about solving inclusion of reports from the top level using / or leaves it empty.

CSP Request Details

None

CSP Rejection Details

None

Potential Workaround

None

Attachments

4

Checklist

hide

Activity

Show:

Mike Taylor December 2, 2024 at 3:06 PM

Released as part of v2.4.0

Charlotte Whitt December 2, 2024 at 10:46 AM

Manual test in FOLIO Snapshot version reporting module (mod-reporting-1.3.0-SNAPSHOT.59) using Chrome

The current issue testing here is about what happens when you leave the “Directory within repository” part of the setting blank or set it to /

All works as expected:
blank:

 

 

And /:

 

 

Thanks a ton

I’ll move the ticket to Awaiting release, and then when released, it can be closed as Done-done

Mike Taylor November 29, 2024 at 3:12 PM

Done and merged to master.

, please allow 24 hours for snapshot rebuild, then QA this.

Done

Details

Assignee

Reporter

Tester Assignee

Labels

Priority

Sprint

Development Team

Thor

Fix versions

Release

Ramsons (R2 2024) Bug Fix

RCA Group

Requirements change

TestRail: Cases

Open TestRail: Cases

TestRail: Runs

Open TestRail: Runs
Created November 28, 2024 at 11:41 AM
Updated December 2, 2024 at 3:06 PM
Resolved December 2, 2024 at 10:46 AM
TestRail: Cases
TestRail: Runs