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

Remove prerender server #597

Merged
merged 19 commits into from
Mar 9, 2019

Conversation

SoftwareSing
Copy link
Member

@SoftwareSing SoftwareSing commented Mar 7, 2019

  • config 移除 production 選項
  • configprerenderServer 的選項移除,改為 prerender,並更改其設置內容
    更新時請直接照此設定
"prerender": {
  "use": true,
  "url": "http://127.0.0.1:3900/"
}

因為prerender server與股市一同開啟可能造成太多不確定問題,因此將其從股市移除,改為另外開啟
移除 production 設定,因沒有其他設定與 productioon 綁定

修改config中 prerender 設定
因不再自行開prerender server,將port的設定值拿掉,使url的設定更簡單
增加 use 設定,可以控制是否要使用prerender功能,false的話則不會執行,BOT會拿到不經任何預處理的頁面

增加 docs 資料夾,放置說明文件

mrbigmouth and others added 17 commits January 13, 2019 00:41
…route

 修正建立新創與新創列表之路由順序衝突的問題
因為prerender server與股市一同開啟可能造成太多不確定問題,因此將其從股市移除,改為另外開啟

移除 production 設定,因沒有其他設定與 productioon 綁定
因不再自行開prerender server,將port的設定值拿掉,使url的設定更簡單
增加 use 設定,可以控制是否要使用prerender功能,false的話則不會執行,BOT會拿到不經任何預處理的頁面
@SoftwareSing
Copy link
Member Author

SoftwareSing commented Mar 8, 2019

先關閉,修改文件與一些問題

SoftwareSing and others added 2 commits March 8, 2019 22:48
prerender不再強制使用https,避免在測試時還必須處理https驗證
會按照BOT訪問時使用的協定
感謝frozenmouse協助修改文件

Co-Authored-By: frozenmouse <frozenmouse@gmail.com>
@SoftwareSing SoftwareSing reopened this Mar 8, 2019
@mrbigmouth mrbigmouth merged commit 494aee3 into ACGN-stock:dev Mar 9, 2019
@SoftwareSing SoftwareSing deleted the remove-prerender-server branch April 19, 2019 07:11
This pull request was closed.
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