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

修改PageDownloadClient.xaml.vb和PageDownloadInstall.xaml.vb #3780

Closed
wants to merge 5 commits into from

Conversation

Silverteal
Copy link
Contributor

对4月1日版本直接判定为愚人节版

根据目前为止的观察结果,这应该不会带来错误的分类,并可以增加更新愚人节版本的缓冲时间

对4月1日版本直接判定为愚人节版
@kong-que-dnf123
Copy link

kong-que-dnf123 commented May 4, 2024

所以至今为止Mojang还是微软没有明确声明四月一日 不会发布正式版(亦或是预览版) 对吗...?

@Silverteal
Copy link
Contributor Author

所以至今为止Mojang还是微软没有明确声明四月一日 不会发布正式版(亦或是预览版) 对吗...?

答案是:据我所知,是,但不太可能发生。

已考虑过这个问题,结论是:
1.此补丁只对版本列表中标记为Snapshot(快照版)的4月1日版本进行标记,不会影响正式版。
2.最坏的结果只是有可能将快照版归类为愚人节版(而且不太可能发生),而现有情况几乎一定会将新的愚人节版归类为快照版。所以这种方法至少不会比当前方法更差
3.愚人节发布的版本就是愚人节版,即使归类错误了,在文字上也能解释得通。
4.如果确实出现了这种情况,和当前版本一样,可通过版本更新来解决,如:限定此补丁只对某年以后的版本生效,同时用现有方法维护愚人节版本列表。

@allMagicNB
Copy link
Contributor

image

@Ad-closeNN
Copy link
Contributor

@Silverteal

@Silverteal
Copy link
Contributor Author

image

此版本的发布时间是3月31日,显示为4月1日是由于使用了UTC+8时区

@allMagicNB
Copy link
Contributor

image

此版本的发布时间是3月31日,显示为4月1日是由于使用了UTC+8时区

代码中是先转 GMT+8:00 再判断,还是先判断最后显示结果时再转 GMT+8:00?

@Ad-closeNN
Copy link
Contributor

Ad-closeNN commented May 4, 2024

image
image
这个是UTC 3月31日1.RV-Pre1
如果使用PR代码(UTC),是不是会变成快照版?

@Silverteal
Copy link
Contributor Author

Silverteal commented May 4, 2024

image
image
这个是UTC 3月31日1.RV-Pre1
如果使用PR代码(UTC),是不是会变成快照版?

不会,因为此版本在单独的愚人节版本列表中。此PR内容只标记4月1日版本为愚人节版,不会把已经标记为愚人节版的版本移出愚人节版

@Silverteal
Copy link
Contributor Author

Silverteal commented May 4, 2024

此外已确定当前写法中识别到的日期是用户的本地日期而非UTC日期,会导致问题,因此不宜直接合并。

我不太会用vb,不太知道怎么改

@LTCatt LTCatt added 处理中 开发者正在调查或处理该项 🟩 低 优先度:低 labels May 5, 2024
@Silverteal Silverteal deleted the branch Hex-Dragon:main June 1, 2024 03:21
@Silverteal Silverteal closed this Jun 1, 2024
@Silverteal Silverteal deleted the main branch June 1, 2024 03:21
@LTCatt LTCatt removed 处理中 开发者正在调查或处理该项 🟩 低 优先度:低 labels Jun 3, 2024
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.

7 participants