修改推送为deploy完成后&&新增设置指定文章不推送 #9
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
昨晚上尝试了一下，我这边修改了部分错误的逻辑 |
Closed
|
没学过JavaScript，都是赶鸭子上架🤣临时学的，感觉变量传递这块和c++，c#有些区别 |
|
我也是赶鸭子上架，本来是小白自用的，看看AI能不能解决这些问题吧，他倒是提了很多意见可以参考 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
1、将获取服务端newpost.json代码与推送代码分离，推送代码放置于deployafter，可以有效避免在延迟推送设置为立即推送时由于deploy耗时过长导致推送先于deploy完成以至于用户收到推送时显示文章不存在
2、新增指定文章不推送。在文章的Front-matter内加入disnotification: true即可跳过该推送，避免老文章修改后推送
代码已经过初步功能性测试，测试结果通过
因时间原因未进行系统性完整测试，可能存在潜在bug（目前暂未发现），建议自行测试一次