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

Android 修复Markdown渲染错误,去除Android Studio相关文件 #127

Merged
merged 3 commits into from
Jun 11, 2022

Conversation

kmod-midori
Copy link
Contributor

去除Android Studio相关文件

按照 https://github.com/github/gitignore/blob/main/Android.gitignore 更新了对应的文件,删除了一些由IDE自动生成和更新的文件,方便后续贡献者。

修复Markdown渲染错误(#117

根据官方文档,和传统的RecyclerView类似,View的实例在滚动的时候是会被重用的,所以不应该在factory里面进行数据的绑定,这个操作应该在update中进行(和传统方式一样分离开),这样应该就可以修复相关问题。

更新构建文件

之前的贡献者把自己的签名密钥写死在了构建文件中,不便于其他开发者进行开发,因此进行了修改(否则完全不能进行构建,调试版本都不行),需要一个key_debug.properties文件,内容类似于如下:

storeFile=/Users/wolf/Documents/com.wh.pushdeer
storePassword=wh.pushdeer
keyAlias=whpushdeer
keyPassword=wh.pushdeer

由于缺少/app/src/main/java/com/pushdeer/os/AppKeys.kt这个文件,我目前没有办法进行测试(除非从APK中提取出来,这个也不算是保密的),但是应该可以修复对应的问题。

@alone-wolf
Copy link
Contributor

抱歉因为能力问题和时间问题,很多东西做得很粗糙,之后我找一下那个appkey文件;感谢你为这个开源项目做出的贡献

@easychen easychen merged commit 20697f7 into easychen:main Jun 11, 2022
@easychen
Copy link
Owner

去除Android Studio相关文件

按照 https://github.com/github/gitignore/blob/main/Android.gitignore 更新了对应的文件,删除了一些由IDE自动生成和更新的文件,方便后续贡献者。

修复Markdown渲染错误(#117

根据官方文档,和传统的RecyclerView类似,View的实例在滚动的时候是会被重用的,所以不应该在factory里面进行数据的绑定,这个操作应该在update中进行(和传统方式一样分离开),这样应该就可以修复相关问题。

更新构建文件

之前的贡献者把自己的签名密钥写死在了构建文件中,不便于其他开发者进行开发,因此进行了修改(否则完全不能进行构建,调试版本都不行),需要一个key_debug.properties文件,内容类似于如下:

storeFile=/Users/wolf/Documents/com.wh.pushdeer
storePassword=wh.pushdeer
keyAlias=whpushdeer
keyPassword=wh.pushdeer

由于缺少/app/src/main/java/com/pushdeer/os/AppKeys.kt这个文件,我目前没有办法进行测试(除非从APK中提取出来,这个也不算是保密的),但是应该可以修复对应的问题。

AppKeys.kt 我加上了,不过出于安全原因隐藏了MiPush的key。换成自己的应该可以测试。

https://github.com/easychen/pushdeer/blob/main/android/app/src/main/java/com/pushdeer/os/AppKeys.kt

@kmod-midori
Copy link
Contributor Author

我个人倒是认为没有隐藏小米推送的AppKey的必要,这个Key只能用于接收消息(要产生安全问题,需要暴露Secret),而且把apk拖进jadx就能直接拿到了。

如果可以直接放在里面,在基于官方服务版本的客户端上会比较方便进行测试。

@easychen
Copy link
Owner

我个人倒是认为没有隐藏小米推送的AppKey的必要,这个Key只能用于接收消息(要产生安全问题,需要暴露Secret),而且把apk拖进jadx就能直接拿到了。

如果可以直接放在里面,在基于官方服务版本的客户端上会比较方便进行测试。

mipush id 我已经更新到最新的代码里边了,应该可供测试了。 https://github.com/easychen/pushdeer/blob/main/android/app/src/main/java/com/pushdeer/os/AppKeys.kt

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.

3 participants