Skip to content

記事URLをblog-entry-{id}.html形式に変更 - #226

Merged
fc2dev merged 10 commits into
fc2blog:masterfrom
uzulla:issue221/change-entry-permalink-to-blog-entry-id-html
Feb 8, 2021
Merged

記事URLをblog-entry-{id}.html形式に変更#226
fc2dev merged 10 commits into
fc2blog:masterfrom
uzulla:issue221/change-entry-permalink-to-blog-entry-id-html

Conversation

@uzulla

@uzulla uzulla commented Feb 4, 2021

Copy link
Copy Markdown
Collaborator

#221

#225 を先にマージしてください

  • App::userURL修正
  • デフォルトテンプレート修正
  • 各種テスト修正

NOTE

fc2blogはcanonicalがないようだが、いくつか見て回ってもないので必須ではないと判断。
(スマホで一部canonicalにするとややこしそうな箇所はある(コメント一覧など))

作業時間 1.2h

uzulla added 10 commits January 31, 2021 21:33


- No longer the current blog_password to read.
- e2e test be fail caused by anchor tag that like`a src="#" onclick="form.submit()"`.
- Puppeteer's waitForNavigation() will be return null when with click() that element, Not return a Response object.
- If use timer for waiting, test will work. but not neat.
- My guessing. When click the a tag  (`src="#"`) by pupp, will be run waitForNavigation on not wanted timing.
- So, cancel a event in onclick event.
It is a runnable, but will be conflicts with blog_crawl.test.ts when run in parallel now(WIP).
Have to decoupling a entry(comment) id.
@fc2dev
fc2dev merged commit 94bc5df into fc2blog:master Feb 8, 2021
@uzulla
uzulla deleted the issue221/change-entry-permalink-to-blog-entry-id-html branch July 31, 2021 03:51
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