Only(?) direct groups used for filtering PRs about trailer

[verebes1]

I noticed a lot of issues since the upgrade from 1.8.9 onwards. Try reverting to that version to see if it will work again.

[ptsochantaris]

@SevereOverfl0w Thanks for the detailed report, and sorry for the slow reply. From the hiding log it does indeed sound like the PR was not promoted to those sections as Trailer was unable to read "All Developers" from the GH API as part of your team memberships. Since I don't have a lot of experience with nested team structures, do you by any chance know where in the GH API this info might exist? I can happily add this capability, if GH offers the info.

[ptsochantaris]

@verebes1 If you create issues for things you have encountered they are far more likely to be noticed and fixed. PRs and/or API info that may be related to issues is also super helpful, if you can provide it as part of an issue.

[SevereOverfl0w]

@SevereOverfl0w Thanks for the detailed report, and sorry for the slow reply. From the hiding log it does indeed sound like the PR was not promoted to those sections as Trailer was unable to read "All Developers" from the GH API as part of your team memberships. Since I don't have a lot of experience with nested team structures, do you by any chance know where in the GH API this info might exist? I can happily add this capability, if GH offers the info.

Just had a look, when you list teams there's a "parent" attribute which tells you the parent of that team: https://docs.github.com/en/rest/teams/teams?apiVersion=2022-11-28#list-teams

You'd need to follow that recursively to build up the whole tree, as parents can have parents, etc.

[ptsochantaris]

Thanks! Do you know of some account that is publicly queryable and which would provide some nested levels of teams? It sounds like a relatively simple issue to fix in Trailer, but I just need to be able to test it with real-world API :)

[SevereOverfl0w]

Do you have a test organisation you use for testing trailer? I believe you should be able to set this up there.