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.

Sign up for GitHub

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

Jump to bottom

开发启动前的待解决问题 #12

Open
5 tasks done
Ayase-252 opened this issue Jan 7, 2022 · 33 comments
Open
5 tasks done

开发启动前的待解决问题 #12

Ayase-252 opened this issue Jan 7, 2022 · 33 comments
Labels
roadmap

Comments

@Ayase-252
Copy link
Contributor

Ayase-252 commented Jan 7, 2022 •
edited
Loading

这个 Issue 列出正式开发前待讨论/待解决的问题：

  • CSS 预处理/后处理：LESS 好了
  • CSS linter？
  • 贡献指南 docs: add contributing guide #26
  • Mock 框架：msw
  • 使用 husky / 配合 husky，增加 eslint、commitlint

欢迎补充
@trim21
Copy link
Contributor

trim21 commented Jan 8, 2022

主站旧的less样式不知道有没有参考价值？

@Sai
Copy link
Member

Sai commented Jan 9, 2022

  • Less，依赖简单

@trim21
Copy link
Contributor

trim21 commented Jan 9, 2022 •
edited
Loading

sass 现在有纯 js 的npm包，也不需要额外下载二进制了

@Sai
Copy link
Member

Sai commented Jan 9, 2022

主站旧的less样式不知道有没有参考价值？

那个 variables.less 估计有点点用

@trim21
Copy link
Contributor

trim21 commented Jan 10, 2022

f6f8580dcc0f4d87 搜索的坑比我预想的大了一些，我们先从条目讨论版开始做吧。然后是小组和章节/角色/人物评论

@shadowdreamer
Copy link
Contributor

react路由用哪个？

@Ayase-252
Copy link
Contributor Author

react-router？ 但是我在想我们要做成 SPA 吗？

@shadowdreamer
Copy link
Contributor

react-router？ 但是我在想我们要做成 SPA 吗？
现在都是spa吧，ssr首屏之后应该也是spa

@ubbcou
Copy link
Contributor

ubbcou commented Jan 18, 2022 •
edited
Loading

待讨论/待解决的问题：

  • 使用 husky
  • 配合 husky，增加 eslint、commitlint
  • taiwindcss

@ubbcou
Copy link
Contributor

ubbcou commented Jan 18, 2022 •
edited
Loading

Roadmap: bangumi/dev-docs#1

@cijiugechu
Copy link
Contributor

需要添加 api mock 吗，比如 msw

@Ayase-252
Copy link
Contributor Author

我看 msw 只支持 GraphQL 和 RESTFul 的API。咱的 API 符合 REST 吗？ @trim21

@trim21
Copy link
Contributor

trim21 commented Jan 18, 2022

msw 这个 Restful api 就是普通的 http api 吧

@trim21
Copy link
Contributor

trim21 commented Jan 18, 2022

设计是尽量按照rest设计的

@Ayase-252
Copy link
Contributor Author

OK, 那开个 issue 加吧

This was referenced Jan 18, 2022
Closed
Closed
@ooPG233
Copy link

ooPG233 commented Jan 18, 2022 •
edited
Loading

待讨论/待解决的问题：

  • 使用 husky
  • 配合 husky，增加 eslint、commitlint
  • taiwindcss

好奇为什么Tailwind划掉了

@ubbcou
Copy link
Contributor

ubbcou commented Jan 19, 2022 •
edited
Loading

待讨论/待解决的问题：

  • 使用 husky
  • 配合 husky，增加 eslint、commitlint
  • taiwindcss

好奇为什么Tailwind划掉了

slack 前端频道里讨论了几句，总之不怎么合适的样子

@shadowdreamer
Copy link
Contributor

还有到底需不需要ssr，架构差挺多的

@Ayase-252
Copy link
Contributor Author

先确定下我们会不会有对应的服务器资源？

@trim21
Copy link
Contributor

trim21 commented Jan 19, 2022 •
edited
Loading

先确定下我们会不会有对应的服务器资源？

有，如果要做的话打包成docker镜像。

@cokemine
Copy link
Contributor

需不需要改协议为 AGPL
bangumi/server#87

@trim21
Copy link
Contributor

trim21 commented Jan 19, 2022

前端没有AGPL的必要吧

@cokemine
Copy link
Contributor

前端没有AGPL的必要吧

如果有其它程序使用了 design 库，在程序打包后。GPL/AGPL 应该都是要求开源未打包前的代码的。

@trim21
Copy link
Contributor

trim21 commented Jan 20, 2022 •
edited
Loading

如果有其它程序使用了 design 库，在程序打包后。GPL/AGPL 应该都是要求开源未打包前的代码的。

如果后端换成了agpl的话我感觉前端用bsd协议就足够了，不过换GPL我也不反对就是了。agpl就没啥必要了。

@cokemine
Copy link
Contributor

如果有其它程序使用了 design 库，在程序打包后。GPL/AGPL 应该都是要求开源未打包前的代码的。

如果后端换成了agpl的话我感觉前端用bsd协议就足够了，不过换GPL我也不反对就是了。agpl就没啥必要了。

我也是无所谓（
AGPL和GPL在这里我觉得是基本没区别的 b38

@cijiugechu
Copy link
Contributor

关于目标平台， IE11 和国产移动端浏览器( UC, 百度 , QQ 这些 )需要考虑兼容吗

@Ayase-252 Ayase-252 mentioned this issue Mar 4, 2022
Open
@trim21
Copy link
Contributor

trim21 commented Mar 20, 2022

还有router和SSR的问题

@trim21
Copy link
Contributor

trim21 commented May 20, 2022

关于目标平台， IE11 和国产移动端浏览器( UC, 百度 , QQ 这些 )需要考虑兼容吗

只考虑chrome，firefox和safari的比较新的版本

@trim21
Copy link
Contributor

trim21 commented Jun 1, 2022 •
edited
Loading

所以怎么发布，ssr还是spa

@Ayase-252
Copy link
Contributor Author

Ayase-252 commented Jun 1, 2022 •
edited
Loading

先 spa 吧，每合入 master 发一次？（不过可能以后有暂时不适合发布的 PR，比如做到一半的需求

@trim21
Copy link
Contributor

trim21 commented Jun 1, 2022 •
edited
Loading

那每次要发版打个tag发release吧。ci打包好了上传到release里。我直接从release的Assets里面拉。从master拉还得我来构建（

@Aloento
Copy link

Aloento commented Jul 17, 2022

spa好，要做前后端分离就没必要做ssr，不然分离的意义不大

@FoundTheWOUT FoundTheWOUT mentioned this issue Oct 24, 2022
Closed
@Innei
Copy link

Innei commented Dec 2, 2022

spa好，要做前后端分离就没必要做ssr，不然分离的意义不大

前提是不考虑搜索引擎抓取和首屏渲染时间。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
roadmap
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
10 participants
@Sai @shadowdreamer @Ayase-252 @ooPG233 @Aloento @trim21 @ubbcou @cijiugechu @Innei @cokemine