Skip to content

35 planner 3 dots - #36

Closed
zhangj23 wants to merge 2 commits into
main-previewfrom
35-planner-3-dots
Closed

35 planner 3 dots#36
zhangj23 wants to merge 2 commits into
main-previewfrom
35-planner-3-dots

Conversation

@zhangj23

Copy link
Copy Markdown
Contributor

No description provided.

@zhangj23
zhangj23 changed the base branch from main to main-preview April 15, 2025 20:02
Ran Prettier check so the PR can be merged
@liug88 liug88 self-assigned this Apr 22, 2025
liug88
liug88 previously approved these changes Apr 22, 2025

@liug88 liug88 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@zhangj23
zhangj23 dismissed liug88’s stale review April 22, 2025 20:27

The merge-base changed after approval.

@liug88 liug88 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

liug88
liug88 previously approved these changes Apr 22, 2025

@liug88 liug88 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A

@zhangj23
zhangj23 dismissed liug88’s stale review April 22, 2025 20:32

The merge-base changed after approval.

ramonechen
ramonechen previously approved these changes Apr 22, 2025
@zhangj23
zhangj23 dismissed ramonechen’s stale review April 22, 2025 21:00

The merge-base changed after approval.

mirmirmirr
mirmirmirr previously approved these changes Apr 23, 2025

@mirmirmirr mirmirmirr left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@zhangj23
zhangj23 dismissed mirmirmirr’s stale review April 23, 2025 17:17

The merge-base changed after approval.

mirmirmirr
mirmirmirr previously approved these changes Sep 16, 2025
@zhangj23
zhangj23 dismissed mirmirmirr’s stale review September 16, 2025 20:39

The merge-base changed after approval.

@mirmirmirr mirmirmirr closed this Sep 19, 2025
@mirmirmirr

Copy link
Copy Markdown
Contributor

This PR seems to be auto-rejecting approvals which prevents changes to be merged to main-preview. I am closing this merge request and creating a new one in its place as an attempt to fix this.

@mirmirmirr mirmirmirr reopened this Sep 19, 2025
@mirmirmirr mirmirmirr closed this Sep 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants