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

Add js and css file and point those links to the site locally. #29

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

CODZR
Copy link
Contributor

@CODZR CODZR commented Apr 16, 2020

我已经解压了css.zip并且将其放在与course.php同级目录下,并将所有原来指向远程网站的src指向本地css/中css或者js文件。 DaiZerong 201732120115

@spm2020spring
Copy link
Member

@dzr201732120115

Thanks, Dai Zerong. I think you changes work.

Could you remove the css folder, and the JS and CSS files inside it? We don't want to include these third party files in the central repo.

Hui

@CODZR
Copy link
Contributor Author

CODZR commented Apr 17, 2020 via email

@spm2020spring
Copy link
Member

好的,意思是把css 和 js 文件拿出来放到与course.php同级嘛,如果是的话,我会在Dai-AddJsLocal(当前页面pull request)更新并提交它。 值得一提的是,我不太明白不想在中央仓库包含这些第三方库文件的意思。 是想像nor/css那样这些文件放在网站的本地并引用 <---还是---> 放在中央仓库中与Course.php同级去引用。 如果是放在网站本地去引用我没有权限,而放在我自己的服务器上或许并不稳定。

------------------ 原始邮件 ------------------ 发件人: "Software Project Management"<notifications@github.com>; 发送时间: 2020年4月17日(星期五) 晚上9:33 收件人: "lanlab-org/LRR"<LRR@noreply.github.com>; 抄送: "975336710"<975336710@qq.com>;"Mention"<mention@noreply.github.com>; 主题: Re: [lanlab-org/LRR] Add js and css file and point those links to the site locally. (#29) @dzr201732120115 Thanks, Dai Zerong. I think you changes work. Could you remove the css folder, and the JS and CSS files inside it? We don't want to include these third party files in the central repo. Hui — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe.

@dzr201732120115

就是(暂时)不要在仓库里添加 css 文件夹以及 css 文件夹里面的东西。

因为里面的JS文件,CSS文件是可以从别处下载的,并不需要我们对它们进行版本管理。

我们要版本管理的文件是我们手动写的东西(比如Php文件),因为这些东西一直在变,所以要对它们进行版本管理。

蓝珲

@CODZR
Copy link
Contributor Author

CODZR commented Apr 17, 2020

我做了我能做的,将这些js和css文件放到与Course.php同级下,然后指向它们,并删除了css文件夹,顺带修改了上次没有指向本地的Submissions.php。
另外一提,现在的master分支中有个error
image
它在master分支下显示错误,并非我的修改导致,它在homepage/style.css第三行

@spm2020spring
Copy link
Member

将这些js和css文件放到与Course.php同级下

@dzr201732120115

不要这么做。谢谢。

蓝珲

@CODZR
Copy link
Contributor Author

CODZR commented Apr 17, 2020 via email

@CODZR
Copy link
Contributor Author

CODZR commented Apr 17, 2020 via email

@spm2020spring
Copy link
Member

spm2020spring commented Apr 17, 2020

那可能是我理解错了,不过这样我确实不知道我应该干嘛,能具体一些吗,我会另外起一个pull request。

@dzr201732120115

发起另外一个Pull Request。

只包含对下面三个文件的修改

  • Course.php
  • Courses.php
  • Header.php

不要包含css文件夹以及css文件夹里面的文件。或许你可以通过在 .gitignore 文件中加入一行 css/ 来达到不包含css文件夹的目的。

蓝珲

@CODZR
Copy link
Contributor Author

CODZR commented Apr 17, 2020 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants