search

nicoxiang / geektime-downloader

极客时间课程下载器，支持下载极客时间专栏/视频课/每日一课/大厂实践/训练营视频
MIT License
622 stars 156 forks source link

Add support for opencourse #121

Closed conbein49 closed 1 year ago

conbein49 commented 1 year ago

Add support for opencourse. With this change, now we can download the course of url: opencourse/intro/ and opencourse/videointro

conbein49 commented 1 year ago

应该可以解决issue：https://github.com/nicoxiang/geektime-downloader/issues/114

nicoxiang commented 1 year ago

你的账号有对应类型的课程吗？本地跑过是 ok 的对吗？因为我自己的账户里没有相应类型的课程，所以不一定好验证。晚点我详细看下代码，先谢谢你的PR~

conbein49 commented 1 year ago

你的账号有对应类型的课程吗？本地跑过是 ok 的对吗？因为我自己的账户里没有相应类型的课程，所以不一定好验证。晚点我详细看下代码，先谢谢你的PR~

都本地验证过了

nicoxiang commented 1 year ago

公开课我发现有这种课比较特殊，type 是 p30，会有问题 云原生时代的 Java

nicoxiang commented 1 year ago

另外为什么叫 Meeting，我看了下 issue 里的那个课程，上面写着可以通过 我的课程-其他 找到这门课，是不是我们也应该归类到其他而不是 Meeting？我也不知道 Qcon 的这个课是怎么找到的，网站上入口也没有。

conbein49 commented 1 year ago

这个是各种会议的，像全球架构师峰会，开发者大会，去过的可以看到会议视频

Nico Xiang @.***> 于 2023年6月22日周四 19:54写道：

另外为什么叫 Meeting，我看了下 issue 里的那个课程，上面写着可以通过 我的课程-其他 找到这门课，是不是我们也应该归类到其他而不是 Meeting？我也不知道 Qcon 的这个课是怎么找到的，网站上入口也没有。

— Reply to this email directly, view it on GitHub https://github.com/nicoxiang/geektime-downloader/pull/121#issuecomment-1602505659, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHPIP6ZOXQKJH62B3EUEHSLXMQW67ANCNFSM6AAAAAAZOKQOGA . You are receiving this because you authored the thread.Message ID: @.***>

conbein49 commented 1 year ago

感觉很难去穷举尽，只能发现再加了

Nico Xiang @.***> 于 2023年6月22日周四 19:49写道：

公开课我发现有这种课比较特殊，type 是 p30，会有问题 云原生时代的 Java https://time.geekbang.org/opencourse/intro/100067401

— Reply to this email directly, view it on GitHub https://github.com/nicoxiang/geektime-downloader/pull/121#issuecomment-1602500048, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHPIP64EZGU6KJJOSCEIWK3XMQWNNANCNFSM6AAAAAAZOKQOGA . You are receiving this because you authored the thread.Message ID: @.***>