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

所有4月1日发布的版本自动视为愚人节版 #3945

Merged
merged 10 commits into from
Jul 28, 2024

Conversation

Silverteal
Copy link
Contributor

之前的pr #3780 因为误操作被关闭了 现在重新提交

@HexDragon-Bot

This comment was marked as resolved.

@Light-Beacon Light-Beacon added the 等待确认 已经过社区确认,等待开发者确认 label Jun 1, 2024
@allMagicNB
Copy link
Contributor

为啥会编译出错……

@allMagicNB
Copy link
Contributor

先关了,我试试能不能 Reopen

@allMagicNB allMagicNB closed this Jun 1, 2024
@allMagicNB allMagicNB reopened this Jun 1, 2024
@allMagicNB
Copy link
Contributor

好吧不能(

@JingHai-Lingyun
Copy link
Collaborator

……?

@LTCatt LTCatt changed the title #3780 重新提交 所有4月1日发布的版本自动视为愚人节版 Jun 1, 2024
@LTCatt LTCatt added 处理中 开发者正在调查或处理该项 🟩 低 优先度:低 and removed 等待确认 已经过社区确认,等待开发者确认 labels Jun 1, 2024
@MCSteve123
Copy link

不是哥们咋还failing呢

@allMagicNB
Copy link
Contributor

不是哥们咋还failing呢

他在别处表示,在等龙猫给他改(

Copy link
Contributor

@wuliaodexiaoluo wuliaodexiaoluo left a comment

Choose a reason for hiding this comment

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

修下问题点

@wuliaodexiaoluo
Copy link
Contributor

wuliaodexiaoluo commented Jul 4, 2024

这边等下 Silverteal#11 合并再合并
@Silverteal

当然有权限直接编辑也可以

EDIT:本地测试可行性验证通过,在没有手动分类的情况下没有正式版或预览版混入,被注释处理的愚人节版本也能正确识别并加入到列表内

之前改的没改回来
为什么要大写啊
败笔也
'所有4月1日发布的版本可视为愚人节版,但不改动版本描述
Select Case Version("releaseDate").Value(Of Date).ToString("MM'/'dd")
Case "04/01"
If Not Version("type") == "special" Then
Copy link
Contributor

Choose a reason for hiding this comment

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

改成 =

Copy link
Contributor

Choose a reason for hiding this comment

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

改成 =

@@ -44,6 +44,14 @@
Version("type") = "special"
Version.Add("lore", GetMcFoolName(Version("id")))
End Select
'所有4月1日发布的版本可视为愚人节版,但不改动版本描述
Select Case Version("releaseDate").Value(Of Date).ToString("MM'/'dd")
Copy link
Contributor

Choose a reason for hiding this comment

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

releaseTime

@@ -555,6 +555,14 @@
Version("type") = "special"
Version.Add("lore", GetMcFoolName(Version("id")))
End Select
'所有4月1日发布的版本视为愚人节版,但不改动版本描述
Select Case Version("releaseDate").Value(Of Date).ToString("MM'/'dd")
Copy link
Contributor

@wuliaodexiaoluo wuliaodexiaoluo Jul 5, 2024

Choose a reason for hiding this comment

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

releaseTime

'所有4月1日发布的版本视为愚人节版,但不改动版本描述
Select Case Version("releaseDate").Value(Of Date).ToString("MM'/'dd")
Case "04/01"
If Not Version("type") == "special" Then
Copy link
Contributor

@wuliaodexiaoluo wuliaodexiaoluo Jul 5, 2024

Choose a reason for hiding this comment

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

单等号
要不然下次写成 is 得了

Copy link
Contributor

@SALTWOOD SALTWOOD left a comment

Choose a reason for hiding this comment

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

两处编译错误,两处逻辑错误。

'所有4月1日发布的版本可视为愚人节版,但不改动版本描述
Select Case Version("releaseDate").Value(Of Date).ToString("MM'/'dd")
Case "04/01"
If Not Version("type") == "special" Then
Copy link
Contributor

Choose a reason for hiding this comment

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

改成 =

'所有4月1日发布的版本视为愚人节版,但不改动版本描述
Select Case Version("releaseDate").Value(Of Date).ToString("MM'/'dd")
Case "04/01"
If Not Version("type") == "special" Then
Copy link
Contributor

Choose a reason for hiding this comment

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

VB 是单等号比较,不是双等号

Copy link
Contributor

Choose a reason for hiding this comment

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

py 写多了可能

@@ -44,6 +44,14 @@
Version("type") = "special"
Version.Add("lore", GetMcFoolName(Version("id")))
End Select
'所有4月1日发布的版本可视为愚人节版,但不改动版本描述
Select Case Version("releaseDate").Value(Of Date).ToString("MM'/'dd")
Copy link
Contributor

Choose a reason for hiding this comment

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

Time

@LTCatt LTCatt merged commit 8f49799 into Hex-Dragon:main Jul 28, 2024
5 checks passed
@LTCatt LTCatt added 完成 已被处理,将在下次更新之后生效 and removed 处理中 开发者正在调查或处理该项 labels Jul 28, 2024
@hejiehao
Copy link
Contributor

hejiehao commented Aug 9, 2024

image
应该使用 Mojang 提供的原始时间而不是转换过后的时间

@JingHai-Lingyun
Copy link
Collaborator

修补代码 于 #4449

@Silverteal Silverteal deleted the patch-1 branch August 21, 2024 06:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
· 优化 完成 已被处理,将在下次更新之后生效 🟩 低 优先度:低
Projects
None yet
Development

Successfully merging this pull request may close these issues.