Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix: fix known bugs in feat_bot #256

Merged
merged 2 commits into from
Jun 7, 2023
Merged

fix: fix known bugs in feat_bot #256

merged 2 commits into from
Jun 7, 2023

Conversation

mmagi
Copy link
Contributor

@mmagi mmagi commented Jun 7, 2023 •
edited
Loading

Whether this PR is eventually merged or not, Sonic will thank you very much for your contribution.

无论此PR最终是否合并，Sonic组织都非常感谢您的贡献。

Checklist

  • The title starts with fix, fea, or doc. | 标题为fix、feat或doc开头。
  • I have checked that there are no duplicate pull requests with this request. | 我已检查没有与此请求重复的拉取请求。
  • I have considered and confirmed that this submission is valuable to others. | 我已经考虑过，并确认这份呈件对其他人很有价值。
  • I accept that this submission may not be used. | 我接受此提交可能不会被使用。

Description

fix known bugs in #243

@mmagi mmagi force-pushed the feat_bot branch 2 times, most recently from 436e742 to f9dd05f Compare June 7, 2023 09:53
@ZhouYixun
Copy link
Member

有几个页面的ui我可能会调整下，到时候你这里可能要合并一下

@mmagi mmagi marked this pull request as ready for review June 7, 2023 11:12
@mmagi
Copy link
Contributor Author

mmagi commented Jun 7, 2023

有几个页面的ui我可能会调整下，到时候你这里可能要合并一下

没问题，我会持续rebase到最新的

ZhouYixun
ZhouYixun approved these changes Jun 7, 2023
View reviewed changes
Copy link
Member

@ZhouYixun ZhouYixun left a comment

Choose a reason for hiding this comment

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

mg

@ZhouYixun ZhouYixun merged commit ebbde60 into SonicCloudOrg:main Jun 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ZhouYixun ZhouYixun ZhouYixun approved these changes

Assignees
No one assigned
Labels
feature fix
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mmagi @ZhouYixun