Skip to content

[Economics] 更新指令系统与配置文件系统 #954

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

Merged
merged 6 commits into from
May 28, 2025
Merged

Conversation

Controllerdestiny
Copy link
Contributor

添加插件

  • 插件已加入解决方案 (Plugin.sln)
  • 插件项目已导入template.targets ()
  • 插件信息已添加至对应manifest.json
  • 插件的文件夹名字和插件的插件项目名字一样 (XXX/XXX.csproj)
  • 添加插件单独的README.md文件 (XXX/README.md)
  • 插件可以正常工作

更新插件/修复BUG

  • 插件已修改版本号
  • 更新插件README.md中的更新日志
  • 插件可以正常工作

其他

  • ❤️熙恩我喜欢你

@THEXN THEXN requested a review from Copilot May 26, 2025 05:19
@Controllerdestiny Controllerdestiny review requested due to automatic review settings May 26, 2025 05:20
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR refactors the Economics plugin into a new Economics.Core module, introduces a unified JSON config system, and streamlines command registration and event handling.

  • Rename EconomicsAPI to Economics.Core across namespaces, projects, and solution references
  • Add JsonConfigBase for loading/unloading settings via Setting.Instance and replace manual reload logic
  • Update command framework, replace GeneralHooks.ReloadEvent usage, and register new tile-edit hook

Reviewed Changes

Copilot reviewed 114 out of 114 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/Economics.Core/Economics.cs Rename namespace, refactor initialization, add OnTileEdit handler
src/Economics.Core/ConfigFiles/JsonConfigBase.cs New abstract base for JSON-based config loading
src/Economics.Core/Command/BaseCommand.cs Revamped subcommand discovery and invocation logic
src/Economics.Core/DB/CurrencyManager.cs Namespace update, expose GetCurrencies API
Plugin.sln Updated project reference from EconomicsAPI to Economics.Core

@Controllerdestiny Controllerdestiny added this pull request to the merge queue May 28, 2025
Merged via the queue into master with commit 00fce1a May 28, 2025
2 checks passed
@ACaiCat ACaiCat deleted the ec branch June 23, 2025 00:16
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.

Economic.RPG Bug Reports and Update Suggestions Economic.API Bug Reports and Update Suggestions Economics RPG translate command
2 participants