Pending

User tests: Successful: Unsuccessful:

avatar weeblr
weeblr
1 Mar 2021

Pull Request for Issue #314 .

Summary of Changes

Use pulls rest API endpoint instead of issues endpoint to fetch the list of PRs. Using issues endpoint caused only a subset of PRs to be retrieved.

Testing Instructions

Applying change make last 100 PR (batchsize) to be listed in the patch tester admin.

avatar weeblr weeblr - open - 1 Mar 2021
avatar weeblr
weeblr - comment - 4 Mar 2021

@roland-d Can you make sure to pull that one in as well? Joomla 3 version has the same problem as the J4 version. Thanks

avatar richard67
richard67 - comment - 4 Mar 2021

@weeblr Maybe you can fix code style before? You have removed an if condition around a code block, so that code block needs one level = one tab less indentation:
https://github.com/joomla-extensions/patchtester/pull/315/files#diff-52387c1e54fadcf3939c94d09d0c914e6eaffd024fe8fd3f47ac4bc289ab01e1R373-R399

avatar weeblr
weeblr - comment - 4 Mar 2021

@richard67 I have attempted to reformat, I'm doing it in Github directly, not the easiest there.

avatar richard67
richard67 - comment - 4 Mar 2021

@weeblr Looks ok here locally with an editor which shows tabs and spaces (if I want that). Not easy in the GitHub UI, I agree. Thanks so far.

I don't know if and when @roland-d 's plans a 3.0.0 release.

avatar weeblr
weeblr - comment - 4 Mar 2021

Well I hope he does cause that's where the problem lies for me and at least on issue solved but pending tests. I have testers, they spent times trying to start using patchtester only to find it would not show the PR I was asking them to test.

Wonder how this has affected the development and testing of Joomla 4 indeed, this has likely been going on for a while now. If people did not see PR in the patchtester, then obviously they could not test them!

avatar roland-d roland-d - change - 4 Mar 2021
Status New Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-03-04 19:21:31
Closed_By roland-d
avatar roland-d roland-d - close - 4 Mar 2021
avatar roland-d roland-d - merge - 4 Mar 2021

Add a Comment

Login with GitHub to post a comment