样式优化,解决有些demo 不可用的问题(天气),解决有些题目失效无法解答的问题(twitch),解决有些工具无法使用的问题(c9.io)#521
Open
ZhichengChen wants to merge 5 commits intoFreeCodeCampChina:devfrom
Open
样式优化,解决有些demo 不可用的问题(天气),解决有些题目失效无法解答的问题(twitch),解决有些工具无法使用的问题(c9.io)#521ZhichengChen wants to merge 5 commits intoFreeCodeCampChina:devfrom
ZhichengChen wants to merge 5 commits intoFreeCodeCampChina:devfrom
Conversation
added 5 commits
May 14, 2018 18:23
c9.io 变化较大,完成课程难度较高 使用多家后选择codeanywhere
fork 计划及已完成功能说明
Contributor
|
好大的一个pr。。我要回家build一下慢慢看看 |
Contributor
|
@ZhichengChen 麻烦问下,之前的 weather app 有什么问题呢?啥时候会无法显示? |
Author
|
@S1ngS1ng 您好,注意到您使用了 ipinfo.io 的api ,很赞,但是在国内环境下还不能访问,而且不再GFWList 默认列表里,所以我必须开启全局vpn 模式,或者手动添加该地址,才能显示(非本地,而是vpn 服务器所在地址的天气),否则天气会无法显示 |
Contributor
|
@Becavalier Consider including this PR for beta release |
Contributor
|
@ZhichengChen 先谢谢你的 PR。由于最近计划做新版本的更新和 beta release,很抱歉我们现在没法 merge 你的代码。一方面 file changes 比较多,另一方面我们需要目前上线的 codebase 来做比较。 目前计划新版本会在放在另一个 branch 做开发,所以你的这个 PR 先不着急关 |
Author
|
好的,感谢您的关注 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
更改了html 课程的样式,预览:http://fcc.chenzhicheng.com/challenges/say-hello-to-html-element
将facebook 群组改为微信群组
/challenges/show-the-local-weather demo 偶尔不能正常显示
/challenges/use-the-twitchtv-json-api 课程说明链接失效
BackEnd 课程里c9.io 改版后变化较大,需要购买亚马逊服务器,完成课程难度较高,试用多家后最终选择codeanywhere,并制作替换了了gif 教程,预览: http://fcc.chenzhicheng.com/challenges/get-set-for-our-back-end-development-projects