Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

群组内一键添加好友 || One-click to add friends to the group #251

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.

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

Open
zhanzhangl wants to merge 2 commits into msgbyte:master
base: master
Choose a base branch
Loading
from zhanzhangl:master

Conversation

@zhanzhangl
Copy link

@zhanzhangl zhanzhangl commented Aug 27, 2024
edited by github-actions bot
Loading

群组内一键添加好友


One-click to add friends to the group

Copy link

netlify bot commented Aug 27, 2024
edited
Loading

👷 Deploy request for tailchat-nightly pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit da42fed

@github-actions github-actions bot changed the title (削除) 群组内一键添加好友 (削除ここまで) (追記) 群组内一键添加好友 || One-click to add friends to the group (追記ここまで) Aug 27, 2024
Copy link
Author

zhanzhangl commented Aug 27, 2024
edited by github-actions bot
Loading

修改密码处添加密码验证规则


Add password verification rules

newPassword: metaFormFieldSchema
.string()
.min(6, t('密码不能低于6位'))
.matches(/[A-Z]/, '密码必须包含大写字母')
Copy link
Contributor

@moonrailgun moonrailgun Sep 1, 2024

Choose a reason for hiding this comment

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

you should wrap with t which can include translation support

<div className="pt-2">{pluginUserExtraInfoEl}</div>

<div className="text-right">
<Tooltip title={t('添加好友')}>
Copy link
Contributor

@moonrailgun moonrailgun Sep 1, 2024

Choose a reason for hiding this comment

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

i think its should also have a permission check like allowSendMessage because its should can be close if some group dont wanna allow user DM


// 一键添加好友
const [, handleAddFriend] = useAsyncRequest(async () => {
const { data } = await request.post('/api/friend/request/add', {
Copy link
Contributor

@moonrailgun moonrailgun Sep 1, 2024

Choose a reason for hiding this comment

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

i think you should use addFriendRequest in client/shared/model/friend.ts

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@moonrailgun moonrailgun moonrailgun requested changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /