Conversation
| late File userDefinedFile; | ||
| late ClassTableData classTableData; | ||
| late UserDefinedClassData userDefinedClassData; | ||
| bool _isClassTableChangedForSystemCalendarSync = false; |
There was a problem hiding this comment.
SystemCalendarSync 您可能需要解释一下？我第一眼难以看懂。
There was a problem hiding this comment.
您好，这里的 SystemCalendarSync 指的是导出到系统日历后的同步
| Future(() => electricity.update()), | ||
| Future(() => school_net.update()), | ||
| ]).then((value) => updateCurrentData()).onError((error, stackTrace) { | ||
| ]).then((value) async { |
There was a problem hiding this comment.
是的，但不是每次刷新都同步。
新课表和本地缓存不同，才认为需要同步
|
差不多是你说的意思，我希望多打点注释。 |
是的，是我代码习惯不好。我抽空补上一些注释。 |
|
看来您得需要等我彻底重构完状态管理后重新修改了…… |
好👌🏻 |
|
您可以继续开发了，改一下冲突，然后回答一下：
|
自动同步先看本地有没有保存过 systemCalendarId，然后只按这个 id 去找，这样就能保证您说的第一个问题。 第二个问题是我考虑不周。如果按照我个人喜好的话，我会倾向于留存旧学期的日历并导出一个新学期的，这个我会在解决冲突的时候顺便实现。 第二次手动输出会找到之前那一个导出日历，删掉再按当前数据重建，目的是保证后续自动同步始终只维护一个目标。我想象不到有什么情景需要两份相同的日历。 |
|
您 review 后就可以 merge 了 |
|
稍后吧，未来一周我会忙别的事情。 |
2 participants
去掉系统日历导出名称里的时间戳后缀，改为固定名称.
系统日历导出不再每次都新建日历，而是复用之前导出的系统日历并重建事件.
课程表刷新后会和缓存对比，如果内容发生变化且之前导出的系统日历还存在，则执行日历同步.
兼容旧的导出日历.