Create
cancel
Showing results for 
Search instead for 
Did you mean: 
Sign up Log in
Celebration

Earn badges and make progress

You're on your way to the next level! Join the Kudos program to earn points and save your progress.

Deleted user Avatar
Deleted user

Level 1: Seed

25 / 150 points

Next: Root

Avatar

1 badge earned

Collect

Participate in fun challenges

Challenges come and go, but your rewards stay with you. Do more to earn more!

Challenges
Coins

Gift kudos to your peers

What goes around comes around! Share the love by gifting kudos to your peers.

Recognition
Ribbon

Rise up in the ranks

Keep earning points to reach the top of the leaderboard. It resets every quarter so you always have a chance!

Leaderboard

Come for the products,
stay for the community

The Atlassian Community can help you and your team get more value out of Atlassian products and practices.

Atlassian Community about banner
4,560,068
Community Members
 
Community Events
185
Community Groups

"Changes Requested" pull request UI needs work

https://bitbucket.org/blog/a-new-status-for-pull-requests-in-bitbucket-cloud-changes-requested

I'm finding the PR UI changes to support this feature to be non-intuitive. Changing the grey background to blue when you click Approve doesn't really convey you've approved the PR, but then the Request changes button goes away, so I guess so?

At least change the button text to "Approved".

Also, I generally Approve PRs and then Merge, and it's annoying that I now have to go dig it out of the ... button menu.

Perhaps if merge conditions are met, display the Merge button?

I'm super happy someone made a new button for us, but don't hide a button that's going to get WAY more use just to make sure we know there's a new button available.

image.png

image.png

1 comment

Brandon Reppert
Atlassian Team
Atlassian Team members are employees working across the company in a wide variety of roles.
Dec 16, 2020 • edited

Hi @Sean Lively, thanks for this feedback.

I agree with the button text changing when the buttons are selected. We'll have a change out shortly for "Approve" to become "Approved" when set and likewise for "Request changes" to become "Changes requested".

We're also considering your feedback for merge button placement for PR reviewers in conjunction with some other feedback we've seen. Our general stance is that merging is a primary action for authors while requesting changes is a primary action for reviewers and there is limited permanent real estate in the title bar, but we're aware that there are many cases where reviewers regularly merge pull requests and we're discussing what the right solution for that will be.

Brandon, thanks for the quick changes on the button.

Sadly, now I get to gripe about the file list change. The list/treeview toggle is cool, but give us an option to put it back into the Details tab. 

Comment

Log in or Sign up to comment
TAGS
AUG Leaders

Atlassian Community Events