feat: neogrp topic use new UI #3
Conversation
anig1scur
force-pushed
the
style
branch
3 times, most recently
from
74201e6
to
a065e91
Compare
mastodon/api.py
Outdated
| @@ -350,7 +350,7 @@ def get_mastodon_application(domain): | |||
|
|
|||
|
|
|||
| def get_mastodon_login_url(app, login_domain, version, request): | |||
| url = request.scheme + "://" + request.get_host() + reverse("users:OAuth2_login") | |||
| url = settings.APP_WEBSITE + reverse("users:OAuth2_login") | |||
There was a problem hiding this comment.
这块不能这么改，这样就只能认证 APP_WEBSITE 一个站点了
There was a problem hiding this comment.
是我在 dev 过程中发现 request.get_host() 老是拿到 localhost 然后认证不成功，我改回去
|
另外， |
|
前端这个改法动作好大....可能最大的难点在于，我不太会前端那一套工具链....后续的维护我感觉会很吃力.....不知道有没有可能仍然是借助django templates那一套继续做.... |
|
@anig1scur 就是，有没有可能主要是在 HTML + CSS 上做美化升级，先不急着引入 React 那一套工具链 |
诶 没注意。。。。 不知道啥时候删掉的 |
超级简单！ 我可以写一个前端 dev readme ，写起来会比 templates 那一套易读、方便很多 |
最开始是打算这样的， 但是在改的过程中发现写模板好痛苦 😭 ... 有想要的交互什么的做起来也麻烦 |
anig1scur
force-pushed
the
style
branch
3 times, most recently
from
0fffc2e
to
a1e17d4
Compare
确实需要有个文档，就写到 README 里吧，感谢 |
|
这两天我本地配置一下环境试试～ |
|
对了，还有个问题是，现在的发 topic 的页面，其实是可以发图片的，复制一个图片，粘贴到文本框里，可以试试 |
嗯嗯 我还没有修改新建 topic 这个页面的 ： https://anig1scur-zany-space-engine-5xw6q9vr6jg376g6-8000.preview.app.github.dev/group/1/new_topic 还是说你是指回复现在可以发图片 ？ （试了下不能也） |
回复不行，只有 new_topic 那个页面可以发图片 |
| @@ -50,6 +50,7 @@ | |||
| "common", | |||
| "users", | |||
| "group", | |||
| "django_react_templatetags" | |||
There was a problem hiding this comment.
这个已经加进去啦 0xasahi/neogroup@673d351 (#3)
本地环境顺利起来了，我本地这两天再 review 一下～ |
|
环境没什么问题了，但准备往线上部署的时候，想到一个问题，现在这种写法，会导致SEO无效吧？ @anig1scur |
|
噢！ 容我研究一下 ssr ，这周我会修改小组主页和 home 页 |
|
我在这个 commit 把 SSR 差不多弄好了
可访问 https://anig1scur-zany-space-engine-5xw6q9vr6jg376g6-8000.preview.app.github.dev/group/topic/1/?comment_id=2 禁用 js 试试看 ssr 需要额外起一个 node server ，大概是这样 cd static_source
node hypernova-bootstrap.js |
|
超赞👍，这个ssr有可能fallback回django吗？必须是node吗
|
|
（我感觉... 你好像一个研究生导师在一步步引导我 improve 这个 commit 😂 ） 目前的 ssr 方案是 https://github.com/marteinn/django-react-polls-with-hypernova-examples 这里的 example 2 ，是没法 fallback 到 django 直接走 client side rendering 的。 ssr service 挂了的话就没法渲染，必须是 node example 1 的话也有 pro & con s ，是一个更具弹性的方案 之前有想过用 next.js 来做把前后端完全分离，但这个动作更大了 |
|
@anig1scur OK，走node也行，那这个serve启动区分dev模式和prod模式吗(确实没怎么用过node，择日我准备部署现在这个PR试试了 |
| @@ -0,0 +1,11 @@ | |||
| // create-react-app requirement | |||
| process.env.NODE_ENV = 'production'; | |||
There was a problem hiding this comment.
这里声明了 env 是 production 所以直接 node hypernova-bootstrap.js 就好 （可能对 node 版本有一定要求? 我也并不太清楚）
|
那我找时间开始写首页和小组首页了 有什么要求建议都可以提 lol |
|
为啥把我的 style branch 删掉了 😂 |
这个删除的是远程branch，没事的，理论上来说一个branch被merge之后，就会删除掉，不影响你下次还用style branch，重新push就好～ |
主要是 topic UI 的改动 （需要我提供一些什么方便 review 吗 ? ）感觉最好是先 pull 看看用线上数据库跑会不会有什么问题
@0xasahi plz when you're OK 👀
我想做一个色调是灰/紫的主题 ！ 好像有点不太官方
然后下面这是设想的 user profile 和 home 页布局