Conversation
Contributor
There was a problem hiding this comment.
嗨——我已经审查了你的更改,一切看起来都很棒!
帮我变得更有用!请在每条评论上点击 👍 或 👎,我会根据你的反馈来改进之后的代码审查。
Original comment in English
Hey - I've reviewed your changes and they look great!
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
Contributor
There was a problem hiding this comment.
Pull request overview
该 PR 旨在改进“售卖(SellProduct)”与“运送委托(DeliveryJobs)”两套 Pipeline:重新处理功能未解锁时的分支,并新增“自动根据当前所在地区选择起始地区”的入口逻辑;同时增强从任意界面进入“地区建设”菜单的兼容性。
Changes:
- 重构 SellProduct / DeliveryJobs 的主入口为带循环的结构,并支持自动选择起始地区。
- 将各地区子流程在完成后回到主循环/地区建设菜单,移除原先的 Finish 结束节点。
- SceneManager:进入“地区建设菜单”时增加“已在地区建设/从二级界面返回”的处理路径。
Reviewed changes
Copilot reviewed 9 out of 9 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| assets/resource/pipeline/SellProduct/Wuling.json | 武陵售卖流程结束后改为回到 SellProductLoop,并支持跳回地区建设菜单 |
| assets/resource/pipeline/SellProduct/ValleyIV.json | 四号谷地售卖流程结束后改为回到 SellProductLoop,并支持跳回地区建设菜单 |
| assets/resource/pipeline/SellProduct/SellCore.json | 移除 SellProductSellAreaFinish 结束节点 |
| assets/resource/pipeline/SellProduct.json | 新增 SellProductLoop/Auto/未解锁识别等主流程重构 |
| assets/resource/pipeline/Interface/SceneMenu.json | SceneEnterMenuRegionalDevelopment 增强:支持已在地区建设、以及从二级界面返回 |
| assets/resource/pipeline/DeliveryJobs/Wuling.json | 武陵运送委托流程结束后改为回到 DeliveryJobsLoop,并支持跳回地区建设菜单 |
| assets/resource/pipeline/DeliveryJobs/ValleyIV.json | 四号谷地运送委托流程结束后改为回到 DeliveryJobsLoop,并支持跳回地区建设菜单 |
| assets/resource/pipeline/DeliveryJobs/PackCargo.json | 移除 DeliveryJobsTaskFinish 结束节点 |
| assets/resource/pipeline/DeliveryJobs.json | 新增 DeliveryJobsLoop/Auto/未解锁识别等主流程重构 |
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
This file contains hidden or 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
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.
Summary by Sourcery
在交付和售卖流水线中更稳健地处理区域解锁状态,并在场景菜单中添加自动选择起始区域的支持。
New Features:
Bug Fixes:
Enhancements:
Original summary in English
Summary by Sourcery
Handle region unlocking state more robustly in delivery and selling pipelines and add support for automatically selecting a starting region in the scene menu.
New Features:
Bug Fixes:
Enhancements: