-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Sg/seq based roaming #2755
base: dev
Are you sure you want to change the base?
Sg/seq based roaming #2755
Conversation
…`getAllMessages` endless loading on group
…2549-seq-based-roaming # Conflicts: # mirai-core-api/src/nativeMain/kotlin/contact/roaming/RoamingMessages.kt
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
好了叫我再 review 以下 (request review)
@Serializable | ||
@SerialName("FullyMessageIdentity") | ||
public class RawFullyMessageIdentity( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Serializable | |
@SerialName("FullyMessageIdentity") | |
public class RawFullyMessageIdentity( | |
@Serializable | |
@SerialName("FullMessageIdentity") | |
public class RawFullMessageIdentity( |
* https://github.com/mamoe/mirai/blob/dev/LICENSE | ||
*/ | ||
|
||
package net.mamoe.mirai.message.source |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
换个包? net.mamoe.mirai.message.source.identity
* https://github.com/mamoe/mirai/blob/dev/LICENSE | ||
*/ | ||
|
||
package net.mamoe.mirai.message.source |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
换个包?
/** | ||
* 用于定位消息的消息凭证符, 附带发送者 id 和消息类型 | ||
* | ||
* @since 2.16.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
不用标 .0
,因为 patch 版本不会加新东西
@NotStableForInheritance | ||
@ApiStatus.NonExtendable | ||
@Serializable(FullyMessageIdentitySerializer::class) | ||
public interface FullyMessageIdentity : MessageIdentity { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
public interface FullyMessageIdentity : MessageIdentity { | |
public interface FullMessageIdentity : MessageIdentity { |
mirai-core-api/src/commonMain/kotlin/message/source/MessageIdentity.serializer.kt
Outdated
Show resolved
Hide resolved
mirai-core-api/src/commonMain/kotlin/message/source/MessageIdentity.serializer.kt
Outdated
Show resolved
Hide resolved
@Him188 2.16 准备发了?怎么移到 2.17 了 |
#2549