Skip to content
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

Show actual weeks 显示实际上课周 #13

Open
chinggg opened this issue Sep 2, 2021 · 1 comment
Open

Show actual weeks 显示实际上课周 #13

chinggg opened this issue Sep 2, 2021 · 1 comment
Labels
enhancement New feature or request feature request A request for a certain feature good first issue Good for newcomers

Comments

@chinggg
Copy link
Contributor

chinggg commented Sep 2, 2021

这个feature请求与你遇到的问题有关吗?请描述这个问题

部分课程比如「形势与政策」和「新生研讨课」并不是每周都上课,希望能够支持在卡片上显示实际上课的周数

描述你期望的解决方案

修改 getPeriods() 中的正则表达式以获取周数,并展示在卡片中

附加内容

完成这项任务后你将会掌握:

  • 正则表达式基础
  • HTML/JS/CSS 基础
  • Vue 基本概念

欢迎参与贡献!

@chinggg chinggg added enhancement New feature or request feature request A request for a certain feature good first issue Good for newcomers labels Sep 2, 2021
@ZKLlab
Copy link
Member

ZKLlab commented Sep 5, 2021

目前问题还比较多,单双周之类的问题比较难调整,考虑重构项目了:

  • 课表不能重课;
  • 时间冲突判断的函数冗余,且依赖课表;
  • 课程数据更新解决冲突的过程比较复杂,不容易改动;
  • 整体占内存较大;
  • 桌面版移动版HTML和CSS冗余;
  • ……

因此重构时考虑:

  • 展示和底层完全允许重课,简化冲突判断、课程数据更新流程,且适应秋季学期变化;
  • 数据放到IndexedDB,尽量不把数据全取出来;
  • 上TypeScript,框架换Vue 3.x或React。

另外原来课程爬虫的环境用不了OpenVPN,现在都是手动爬取上传,准备迁移到Github Actions。

@chinggg chinggg added the hacktoberfest hacktoberfest 2022 label Oct 3, 2022
@chinggg chinggg removed the hacktoberfest hacktoberfest 2022 label Feb 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request feature request A request for a certain feature good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants