New Idea

Add setting to allow users to UNCANCEL Reviews

Status: New Idea
by Staff on ‎01-03-2018 01:48 PM

Submitted on behalf of Microsemi (Formerly PMC-Sierra)

 

The motivation for us stems from the cause that usually even project admins have to contact system admins to uncancel a review. It'd be great if there was a feature enhancement for group admins to be able to uncancel a review.

 

Maybe add a new setting that will allow, "Everyone | Group admins | Author | Creator", to UNCANCEL reviews.

 

 

Prevent Completing Reviews that fail to merge in Bitbucket

Status: New Idea
by kevin-m-b ‎01-10-2018 03:04 PM - edited ‎01-10-2018 05:27 PM

My team uses Pull requests within Bitbucket Server and have configured Collaborator to merge the pull request when a review has completed. Unfortunately we have found that if the merge fails, the review still successfully completes.

 

When this occurs we need to:

  1. manually re-open the review
  2. have the developer resolve the problem that prevented the merge from completing (fix the CI build, fix the merge conflict, etc)
  3. re-approve the review and send-to-completed again

The Bitbucket plugin will then re-attempt the merge.

 

When attempting to close a review, Collaborator should wait for Bitbucket to attempt the merge and if it fails, the review should remain open and all review participants should be notified.

Plastic integration

Status: Community Feedback Requested
by Staff ‎12-14-2017 06:03 AM - edited ‎12-14-2017 11:05 AM

Submitted on behalf of Nigel Hoyland (Surgical Innovations).

 

The Plastic SCM integration is requested. 

Filter defects by user

Status: New Idea
by marke on ‎12-21-2017 04:25 PM

It would be useful to be able to filter open defects in the review materials screen by participant. At a minimum, another filter button that filters down to your own open defects. Or more general, a drop-down to select a participant to apply to any of the other filters. I realize you can sort in the review summary screen but having a facility to quickly navigate through your own open defects in the review materials screen would be very useful.

 

Department field in the Users list

Status: New Idea
by Staff on ‎12-14-2017 06:19 AM

Submitted on behalf of Gadadhar Dalei (Nokia).

 

The Admin -> Users list should also contain the user's department values.

Text in the chat window will cut words in half while wrapping the text, making chat text hard to read.  (screen shot attached).

 

Words should wrap at a space break, rather than in the middle of words.

 

This would make read

ing test a lot easier wit

hout the reader havi

ng to re-interpret all th

e broken words

 

Smiley Happy  

When a user attempts to upload a file that is in the "restricted file types" list (a .SH file for example), Collaborator does not indicate to the user that the file was blocked, causing confusion. 

 

The Collaborator server should indicate to the user that some of the files were blocked because they are on the restricted file list.

Allow to pre-fill the 'Before' and 'After' branches in Code Collaborator Client

Status: New Idea
by Thomas_Andre ‎06-28-2017 02:49 AM - edited ‎06-28-2017 03:13 AM

Hello,

 

The Collaborator Enterprise Client's "Add Git Diffs to Review" modal window allows to specify Diffs to review by selecting the Before branch and After branch. We can browse thru existing branches thanks to the drop-down lists (see attached snapshot), but this can quickly become inconvenient when the number of branches increases. There are also some really irritating cases :

 

  1. When you add Git Diffs to existing reviews and have to select the same branches over and over again.

Solution > Persist the latest selected branches for a given existing review and user and allow to have the branches pre-filled

 

  1. When your DevOps guidelines require you to create reviews against a 'Develop' branch systematically. Therefore, you always have to set the 'Before' branch on this 'Develop' branch.

Solution > Configure a default 'Before' branch so that it can automatically be picked up for new reviews.

 

Thanks

Thomas

 

 

 

Allow reviewers to quickly filter the file list by file type

Status: New Idea
by Thomas_Andre ‎06-28-2017 02:20 AM - edited ‎06-28-2017 08:10 PM

Hello,

 

Our Dev team can create some reviews containing a lot of various files (eg. *.cs, *.xml, *.csproj, *.config...).

When reviewing the files under the 'Review Materials' section, we would like to filter the files by their extension types so that we can focus on the most important changes.

 

For instance, we can have just a few critical changes located under some .cs files and many unimportant changes in our .csproj files. In such cases, we would like to see at one glance the modified .cs files.

 

Thanks

Thomas

Sometimes, you want a different person to look at something, or respond to a comment.  My team would like Collaborator to have a capability to tag another user.  For example, in Jira, you can use "@lastname,firstname" to tag another user.  We would like a similar capability in the comment boxes.

One fine summer day, "Jane Smith" (sAMAccountName/username == "janesmit") gets married, and changes her name to Jane Johnson.  After returning from her honeymoon and going back to work, she logs into her computer and other applications as "janejohn", and all is well.  

Unfortunately, when she logs into her company's instance of Collaborator (which uses LDAP Auth) as "janejohn", it is as if she has an entirely new account - the old account "janesmit" is separate, and she no longer has access to the reviews associated with her old account ("janesmit").  Collaborator does not act appropriately when accounts are renamed.

 

Jane is understandably not happy, and writes an email to the maintainers of Collaborator at her company, asking for the accounts to be merged.  Unfortunately, Collaborator provides no such capability to the administrators of the tool, and Jane is sad.  If only SmartBear would add the ability for an administrator to merge accounts, Jane would be happy.  Smiley Happy

 

User experience would be greatly improved if the reviewer could associate their comment or defect to a range of lines instead of just a single line.

 

Additionally, if that range could also be inclusive of characters within a single line that would also be appreciated.  When comments or defects are part of a large block of code being able to accurately call out an area of code instead of just a top line of code is helpful.

Collaborator 11

Status: New Idea
by Normd on ‎06-14-2017 08:29 AM

Remove host/ip info from messages sent to users as it is in my opinion useless info and can be a security concern.

 

In our instance, the server is behind a reverse proxy. Users only know the url. The host info is registered in DNS to the proxy.

Sending the actual server's host info to the user on a password reset is meaningless and negates the purpose of the proxy to hide the actual server from the end users.

 

   

As of Collaborator 11.0.11000, syntax highlighting can be added for previously unsupported languages and existing languages can be modified. 

It's a lot of work to figure out all the regexes for the different things that should be matched though, so it would useful if the community could share their work with each other on languages that aren't yet supported (e.g. Makefiles). 

 

Add the ability to import/export a single language syntax highlighting configuration from the syntax highlighting screen in the Administrative Settings so the whole installation's configuration doesn't need to be exported and cut to share, and exported, modified, and re-imported to use new syntax highlighting settings.

Currently, when one creates issues and then sends to rework, the participant shows as "Approved" in the Participants module, which is highly confusing to our team.

Obviously the review did not approve this review as s/he submitted issues and sent to rework. The participant status should be "Waiting" or some new status "Reviewed", but "Approved" doesn't make sense.

Submitted on behalf of Debdas Mondal (HSBC).

 

GitHub pull requests may have reviewers assigned to them. It should be possible to synchronize the list of pull request reviewers and the list of a linked Collaborator review's participants.

Submitted on behalf of Gadadhar Dalei (Wipro Limited).

 

Currently, Collaborator loads all users to the User List. If there are thousands of users, it may take considerable amount of time.

Submitted on behalf of Avinash Ravikumar (Wipro Limited).

Make Collaborator work with LabView

Status: New Idea
by CarolMonzillo on ‎09-12-2017 06:10 AM

Currently doing Peer Reviews with LabView does not work.  The users would like to be able to put comments on the LabView diagrams.  Saving each file to pdf to be able to comment on each is not a great solution.  There can be many files and files are coordinated with each other.

Suggest Product Idea
We need your help! This is your space to provide suggestions for features or improvements for SmartBear products. Let us know what you think, or how we can help improve your experience with Collaborator.
Top Kudoed Authors