Skip to content

Conversation

@MiyakoMeow
Copy link
Contributor

No description provided.

@MiyakoMeow MiyakoMeow marked this pull request as ready for review November 6, 2025 18:03
@MiyakoMeow
Copy link
Contributor Author

屏幕截图 2025-11-07 020343

Tested.

@Catizard
Copy link
Owner

Catizard commented Nov 7, 2025

what's the reason for doing so?

@MiyakoMeow
Copy link
Contributor Author

准备在难度表那里加一个“添加预置难度表”的功能。
还是说我要一次做完?

@Catizard
Copy link
Owner

Catizard commented Nov 7, 2025

你可以简单的把初始化的页面带过来而不是修改已有的配置,我觉得没必要。真正麻烦的地方是那个接口本身可能没处理重复的问题

@MiyakoMeow MiyakoMeow marked this pull request as draft November 7, 2025 12:56
@MiyakoMeow MiyakoMeow marked this pull request as ready for review November 7, 2025 20:39
@MiyakoMeow MiyakoMeow changed the title fix(setup): move out pre-defined tables into json feat(table): add load-predefined tables function Nov 7, 2025
@MiyakoMeow
Copy link
Contributor Author

已经测试,能够正常运行

@Catizard
Copy link
Owner

现在我认为把predefined的难度表移到json文件是一个改进,但是添加预置难度表的功能我认为简单的把页面带过来不够,需要补齐回显,标题这些问题。如果你把这个pr后续的提交revert掉的话我可以合并进来到一个新分支里

@MiyakoMeow MiyakoMeow force-pushed the move-out-pre-defined-tables branch from 6fc45e1 to 196ec1c Compare November 30, 2025 11:20
@MiyakoMeow
Copy link
Contributor Author

喵喵喵,rebase了喵

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