Contributions
P4 Protects Should Hide File Comments Along With File Content
Currently, you can use aP4 Protects script withCollaborator to prevent users from seeing file content on certain branches. This is helpful from a security point of view, but it doesn't encompass everything. File comments should also be hidden on these files as well. Without this, developers have to worry about directly addressing file contents for fear of incidentally exposing parts of the code to people who don't have access to the content.1.4KViews0likes0CommentsChange Line Number that a Comment Refers To
When we have rteviews with lots of comments and multiple uploads, the original line number that a discussion starts on can get very off very quickly. It would be helpful if either the original commenter or the author could change the line number that a discussion refers to so comments still reference the correct area of the code.1.9KViews2likes3CommentsAdd Ability to Choose File Upload Order
We frequently have code changes that effect a few dozen files, but the bulk of the relevent code changes are only in 2 or 3 of those files. It would be helpful to manually choose the display order of the files to make it more obvious what reviewers should focus on.2.7KViews2likes1CommentHave Collaborator end the process thread if the certificate is not accepted
Currently, if Collaborator makes a request and the request cannot be completed because the certificate is not accepted, it waits for the user to respond. When these requests are made from scripts and there is no user to provide input, then the process simply hangs forever. It would be preferred if the request terminated instead of waiting and there was a specific certificate acceptance command instead.2.8KViews0likes1Comment