Skip to content

插件市场 API 支持#411

Merged
dmzz-yyhyy merged 9 commits into
refactoringfrom
plugin_store_api
Apr 29, 2026
Merged

插件市场 API 支持#411
dmzz-yyhyy merged 9 commits into
refactoringfrom
plugin_store_api

Conversation

@yukonisen

Copy link
Copy Markdown
Collaborator

这个 PR 新增了插件市场 API 的支持,支持从 URL scheme 进入软件并展示插件信息和下载。

修复了一处插件安装时可能导致占用大量空间的逻辑错误。

@dmzz-yyhyy dmzz-yyhyy left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

使用翻译管理平台管理翻译之后必须在平台上提交除英文外的语言
因为语言同步原则上是单项的,除英文外是由平台同步到github,github上的其他语言是从平台同步过来的,如果直接在代码里改语言的话会被平台的提交覆盖掉的

import java.util.concurrent.TimeUnit
import javax.inject.Inject

sealed interface StoreInstallState {

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

能不能把这种类单独放一个文件啊。。。

import javax.inject.Inject
import javax.inject.Singleton

data class PluginUpdateInfo(

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

能不能把这种类单独放一个文件啊。。。

val errorMessageMap: Map<String, String> = pluginManager.errorPluginMap
val scannedPluginApps: List<PluginAppInfo> = pluginManager.appPluginInfos

val pluginUpdates = pluginUpdateCheckRepository.updates

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

这里最好写成val pluginUpdates get() = pluginUpdateCheckRepository.updates

@yukonisen yukonisen requested a review from dmzz-yyhyy April 27, 2026 16:35
# Conflicts:
#	app/src/main/kotlin/indi/dmzz_yyhyy/lightnovelreader/ui/home/settings/pluginmanager/PluginManagerScreen.kt
@dmzz-yyhyy dmzz-yyhyy merged commit 530c00a into refactoring Apr 29, 2026
3 of 4 checks passed
@yukonisen yukonisen deleted the plugin_store_api branch May 5, 2026 16:09
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.

2 participants