Patrick Connolly

Results 485 comments of Patrick Connolly

No objection from me, but to be clear, I don't have merge access directly. I can't imagine the polis team would have issue so long as the solution is scoped...

Hi :) Not sure if this is still relevant, but just an FYI that when a run fails, docker container logs are printed that could give a hint as to...

An underlying assumption of the above docker approach is that we're open to supporting the use-case of someone _eventually_ running a production deploy with `docker-compose`. (Seems to have worked well...

@colinmegill in https://github.com/compdemocracy/polis/discussions/1315#discussioncomment-2200926 > We have had this feature in moderation view previously. It didn't get a lot of use! Your use case is the exact reason we did build...

Any loss to just merging this tiny fixup, and leaving the other fixup for later? workflow might still fail, but clearer error, and one less lingering PR :)

Note: **merging with a merge commit** will make accepting the subsequent PRs easier, so please consider avoiding rebase-merge and squash-merge for this PR.

In your thinking, would it just be for aesthetic in the UI, or is it fetching from API via pagination and multiple calls? ### Current interface

Andy Paice of MutualGain raised a question in the user group discord: ![image](https://user-images.githubusercontent.com/305339/150416435-621fedc0-00ad-4ce5-8cc1-7f56485b9258.png) Does this 100 comment limit come into play on "Accepted" and "Rejected" tabs as well?

Thanks! Will review the rest later, and see about those failing tests One minor thing to note is the `depends_on` key is for the start order of containers, and not...

> The E2E test failures were similar to those I saw on my build machine. However, I got the same errors with dev. I suspect the errors in CI and...