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

建议新增站内谱面上传入口 #15

Closed
electron-shaders opened this issue Mar 9, 2022 · 15 comments
Closed

建议新增站内谱面上传入口 #15

electron-shaders opened this issue Mar 9, 2022 · 15 comments
Labels
体验增强/Enhancement 新功能提交或改进代码/New feature or request 体验问题/Experience problems 此问题影响用户体验/This issue affects user experience stale 正在调查/Under investigation 正在调查此错误发生的原因/Investigating why this error occurred

Comments

@electron-shaders
Copy link
Contributor

现在上传谱面的方式是跳转到github repo进行手动fork+pr，对于没用过github的人来说门槛较高，

因此可以通过在站内搭建一个上传页解决该问题。同时，上传页可以提供预检验功能，避免不必要的commit。

上传操作可考虑使用github oauth app + github api实现。

PS：如果可行的话我可以摸鱼帮忙贡献代码
@yuameshi
Copy link
Owner

yuameshi commented Mar 9, 2022

虽然但是，这样的话多半需要暴露OAuth APP的api key（严重可能还要暴露我自身的TOKEN），否则就需要后端服务器来支持，或者使用cloudflare pages

@yuameshi yuameshi added 体验增强/Enhancement 新功能提交或改进代码/New feature or request 体验问题/Experience problems 此问题影响用户体验/This issue affects user experience 正在调查/Under investigation 正在调查此错误发生的原因/Investigating why this error occurred labels Mar 9, 2022
@yuameshi
Copy link
Owner

yuameshi commented Mar 9, 2022

你要说做一个入口，那肯定是能做的，只是没服务器储存而已

@Dreamail
Copy link

Dreamail commented Mar 9, 2022

虽然但是，这样的话多半需要暴露OAuth APP的api key（严重可能还要暴露我自身的TOKEN），否则就需要后端服务器来支持，或者使用cloudflare pages

存本地cookie（

@yuameshi
Copy link
Owner

yuameshi commented Mar 9, 2022 •
edited

虽然但是，这样的话多半需要暴露OAuth APP的api key（严重可能还要暴露我自身的TOKEN），否则就需要后端服务器来支持，或者使用cloudflare pages

存本地cookie（

我个人认为，api key没有必要，也不应该储存于客户端

@electron-shaders
Copy link
Contributor Author

oauth app的安全问题可以用阿里云的函数计算解决，按量付费每月前100万次请求免费。

@yuameshi
Copy link
Owner

yuameshi commented Mar 9, 2022

oauth app的安全问题可以用阿里云的函数计算解决，按量付费每月前100万次请求免费。

对于这个问题，先开发好上传界面再说吧，我们人还没开发好呢（

@ghost
Copy link

ghost commented Mar 10, 2022

oauth app的安全问题可以用阿里云的函数计算解决，按量付费每月前100万次请求免费。

对于这个问题，先开发好上传界面再说吧，我们人还没开发好呢（

最好还是让谱面作者自己上传，因为挺怕攻击的，我觉得最好的方法就是发一个教程图片吧

@ghost
Copy link

ghost commented Mar 10, 2022

oauth app的安全问题可以用阿里云的函数计算解决，按量付费每月前100万次请求免费。

对于这个问题，先开发好上传界面再说吧，我们人还没开发好呢（

顺带一提，我要准备开始学JavaScript了，毕竟一直提供tip也不是啥好的贡献方式qwq

@yuameshi
Copy link
Owner

oauth app的安全问题可以用阿里云的函数计算解决，按量付费每月前100万次请求免费。

对于这个问题，先开发好上传界面再说吧，我们人还没开发好呢（

顺带一提，我要准备开始学JavaScript了，毕竟一直提供tip也不是啥好的贡献方式qwq

建议学ts

@electron-shaders
Copy link
Contributor Author

做了个界面出来，还没写逻辑（
image

如果还不错的话我就接着干了？

@yuameshi
Copy link
Owner

做了个界面出来，还没写逻辑（
image

如果还不错的话我就接着干了？

其实我们好像有人在干了

@electron-shaders
Copy link
Contributor Author

其实我们好像有人在干了

他进度怎么样了？如果我这版不错的话我就继续写了

@yuameshi
Copy link
Owner

我问问

@electron-shaders
Copy link
Contributor Author

image

完善了下自适应，现在界面弄好了（大概

@electron-shaders electron-shaders mentioned this issue Mar 14, 2022
Closed
@github-actions
Copy link

This issue is marked as stale because it has not had recent activity. Issues marked with stale will be closed if they have no activity within 3 days.

@github-actions github-actions bot added the stale label Mar 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
体验增强/Enhancement 新功能提交或改进代码/New feature or request 体验问题/Experience problems 此问题影响用户体验/This issue affects user experience stale 正在调查/Under investigation 正在调查此错误发生的原因/Investigating why this error occurred
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@Dreamail @electron-shaders @yuameshi