Skip to content
This repository has been archived by the owner on Apr 12, 2023. It is now read-only.

重写一下Action的构建以及增加公告获取失败时的提示与UI优化 #10

Merged
29 commits merged into from
May 17, 2022
Merged

重写一下Action的构建以及增加公告获取失败时的提示与UI优化 #10

29 commits merged into from
May 17, 2022

Conversation

misaka10843
Copy link
Contributor

此pull为测试,因为无法获取到该项目的原pfx(也就是签名文件)
所以在跑到Decode the pfx步骤时会100%停止运行

因此需要在GitHub中的环境变量中添加pfx的base64编码

@misaka10843 misaka10843 marked this pull request as draft May 16, 2022 02:41
@misaka10843
Copy link
Contributor Author

获取pfx的base64编码应该要按照以下步骤进行:

1.在VS的项目设置中生成pfx文件
2.dotnet tool install -g dotnet-base64然后base64 -f 证书名称.pfx -o base64.txt
3.复制base64.txt中后打开 GitHub 的项目的设置,进入到 Secrets 页面,点击 Add a new secret 新建保密信息,此时输入 Base64_Encoded_Pfx 作为保密信息名,内容请填写刚才转换的 base64 的内容即可

@misaka10843
Copy link
Contributor Author

获取pfx的base64编码应该要按照以下步骤进行:

1.在VS的项目设置中生成pfx文件 2.dotnet tool install -g dotnet-base64然后base64 -f 证书名称.pfx -o base64.txt 3.复制base64.txt中后打开 GitHub 的项目的设置,进入到 Secrets 页面,点击 Add a new secret 新建保密信息,此时输入 Base64_Encoded_Pfx 作为保密信息名,内容请填写刚才转换的 base64 的内容即可

似乎可以直接跳过设置证书环节,但是会导致第一次启动跳出蓝框提示

@misaka10843 misaka10843 reopened this May 16, 2022
@misaka10843
Copy link
Contributor Author

在物理机中测试通过,可以进行合并
但是有未知原因导致Resources中的背景图片无法生成到exe中

@misaka10843 misaka10843 marked this pull request as ready for review May 16, 2022 07:07
Copy link
Contributor Author

@misaka10843 misaka10843 left a comment

Choose a reason for hiding this comment

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

GenshinImpact_Lanucher.csproj中将生成中的报错等级调整为0,防止GitHub在构建后显示,可以适量调高此等级

@misaka10843 misaka10843 changed the title [草案]试着修复一下Action中的配置问题 试着重写一下Action的构建问题 May 16, 2022
@misaka10843 misaka10843 changed the title 试着重写一下Action的构建问题 重写一下Action的构建以及增加公告获取失败时的提示而不是崩溃 May 16, 2022
@misaka10843
Copy link
Contributor Author

增加公告获取失败时的提示而不是崩溃
图片

@misaka10843 misaka10843 changed the title 重写一下Action的构建以及增加公告获取失败时的提示而不是崩溃 重写一下Action的构建以及增加公告获取失败时的提示与UI优化 May 16, 2022
@misaka10843 misaka10843 marked this pull request as draft May 16, 2022 10:36
@misaka10843 misaka10843 marked this pull request as ready for review May 16, 2022 11:19
@misaka10843
Copy link
Contributor Author

UI修改如下:
调整两个地方的UI居中问题
修复服务器中右键删除服务器的icon问题
调整公告中鼠标悬停显示黑色文字为白色
提示框统一规范

@BlameTwo BlameTwo closed this May 17, 2022
@BlameTwo BlameTwo reopened this May 17, 2022
@BlameTwo
Copy link
Owner

请去dev分支进行提交

@BlameTwo BlameTwo closed this May 17, 2022
@ghost
Copy link

ghost commented May 17, 2022

请去dev分支进行提交

似乎dev分支是落后于main的

@ghost ghost reopened this May 17, 2022
@ghost ghost merged commit 37b37e9 into BlameTwo:master May 17, 2022
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants