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.

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

feat: add session.fid as user context identifier #56

Merged
merged 2 commits into from
Dec 23, 2022
Merged

feat: add session.fid as user context identifier #56

merged 2 commits into from
Dec 23, 2022

Conversation

Seidko
Copy link
Contributor

@Seidko Seidko commented Dec 19, 2022

内容描述

Session中加入了一个fid(full id)。类似于cidgid以及sid

希望解决的问题

作用是创建一个与平台,群组,用户都相关的标识符。可以做到不同平台,可以做到同一个平台,同一个用户在不同群组中也有不同的id。

对一些服务(比如chatgpt-bot的上下文)很有用。

内容格式

get fid() {
  return `${this.platform}:${this.channelId}:${this.userId}`
}

@Seidko Seidko changed the title chore: added bid chore: added fid Dec 19, 2022
@Seidko
Copy link
Contributor Author

Seidko commented Dec 19, 2022

更名为fid (full id)

@Seidko Seidko changed the title chore: added fid feat: add fid Dec 19, 2022
@Seidko Seidko changed the title feat: add fid feat: add session.fid as user context identifier Dec 19, 2022
Copy link
Contributor

@shigma shigma left a comment

Choose a reason for hiding this comment

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

This still sounds a little wield, but I can't think of a better word so just let it go. 🤣

@shigma shigma merged commit c3a73ba into satorijs:master Dec 23, 2022
@Seidko Seidko deleted the add-bid branch December 26, 2022 08:02
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.

2 participants