noriaki/bus-time-table-web

bus-time-table-web's Issues

Test issue

hoge

発車タイマーのバス停名部分のマージンが無い

SSIA

前発/次発をダブルタップした際に画面が拡大しないようにする

SSIA

各バス停の時刻表を1画面内にすべて表示する

Depends on #13

• 現時刻から数時間分のみ表示し1画面に収める
• 表示部分前後がタップで開閉できる

検索エンジン用のメタデータ(JSON-LD)を埋め込む

SSIA

Fix bug caused by incorrect timezone handling

応急処置として、ビルドコマンドにタイムゾーン環境変数を付け、Vercel上でも日本時間でビルドする。

  "build": "TZ=\"Asia/Tokyo\" next build",

恒久対応は、dayjsのtimezoneを正しく利用して実装する必要がある。
これをテストする方法としては、テスト時のタイムゾーンを日本時間以外に設定してテスト実行する。

  "test:tz:jp": "TZ=\"Asia/Tokyo\" jest",
  "test:tz:foreign": "TZ=\"Etc/GMT\" jest",

Action Required: Fix Renovate Configuration

There is an error with this repository's Renovate configuration that needs to be fixed. As a precaution, Renovate will stop PRs until it is resolved.

Error type: undefined. Note: this is a nested preset so please contact the preset author if you are unable to fix it yourself.

時刻表 (/timetable) と概要 (/info) で最上部へ戻るボタン

• ページ内アンカーのあるコンテンツ部分に入ったら表示
• 右下にフローティングボタン。そんなに目立たなくて良い

本番デプロイ時にステージングとして起動してしまうデプロイスクリプトのバグを修正する

SSIA

ホーム画面追加のお知らせをしばらくしたら小さくする

• アニメーションしてアイコンボタン化
• 初回のみフルサイズからアニメーションし、2回目以降は最初からアイコンボタンで表示する

時刻表を直近4時間分表示に変更すると発車タイマーからのリンク位置がずれる

SSIA

次発以降を表示したときでも「現時刻に対する次発（先発）」が時刻表で分かるようにする

SSIA

Replace time management package moment to dayjs

タイマーの画面にヒントを表示する

ヒントの内容案

• ページ切替は下部ボタン
• 現在表示中ページの下部ボタンタップでページ最上部へスクロール
• 前発・次発ボタンのダブルタップで直近バスと最終バスのタイマー表示

Dependency Dashboard

This issue lists Renovate updates and detected dependencies. Read the Dependency Dashboard docs to learn more.

Rate-Limited

These updates are currently rate-limited. Click on a checkbox below to force their creation now.

• Update all patch dependencies (@types/node, prettier, superjson)
• Update babel monorepo to v7.21.8 (@babel/core, @babel/eslint-parser)
• Update dependency @cybozu/eslint-config to v19
• Update dependency babel-plugin-module-resolver to v5
• Update dependency typescript to v5
• 🔐 Create all rate-limited PRs at once 🔐

Open

These updates have all been created already. Click a checkbox below to force a retry/rebase of any.

• Update dependency next to v12 [SECURITY]
• Update react monorepo (@types/react, @types/react-dom)
• Update mui monorepo (@material-ui/core, @material-ui/icons, @material-ui/styles)
• Update Node.js to v18 (node, @types/node)
• Update actions/checkout action to v3
• Update actions/setup-node action to v3
• Update dependency @testing-library/react-hooks to v8
• Update dependency eslint to v8
• Update jest monorepo to v29 (major) (@types/jest, jest, ts-jest)
• Update react monorepo to v18 (major) (@types/react, @types/react-dom, react, react-dom, react-test-renderer)
• Click on this checkbox to rebase all open PRs at once

Detected dependencies

github-actions

.github/workflows/ci.yml

• actions/checkout v2
• actions/setup-node v2

npm

package.json

• @holiday-jp/holiday_jp 2.4.0
• @material-ui/core 4.11.4
• @material-ui/icons 4.11.2
• @material-ui/styles 4.11.4
• babel-plugin-module-resolver 4.1.0
• clsx 1.2.1
• dayjs 1.11.7
• next 10.2.3
• next-workbox-webpack-plugin 2.1.1
• react 17.0.2
• react-dom 17.0.2
• superjson 1.12.2
• unstated-next 1.1.0
• @babel/core 7.21.3
• @testing-library/react-hooks 5.1.3
• @babel/eslint-parser 7.21.3
• @cybozu/eslint-config 14.0.3
• @types/jest 26.0.24
• @types/node 14.18.41
• @types/react 17.0.44
• @types/react-dom 17.0.15
• @wojtekmaj/enzyme-adapter-react-17 0.8.0
• babel-plugin-superjson-next 0.4.5
• enzyme 3.11.0
• eslint 7.32.0
• eslint-import-resolver-babel-module 5.3.2
• jest 26.6.3
• jest-enzyme 7.1.2
• npm-run-all 4.1.5
• prettier 2.8.7
• react-test-renderer 17.0.2
• ts-jest 26.5.6
• typescript 4.9.5
• node 14.x

---

• Check this box to trigger a request for Renovate to run again on this repository

アプリアイコンにTypoがある

mistake) DUEX TOURS
correct) DEUX TOURS

前発/次発ボタンのラベル文字サイズが小さすぎる

SSIA

material-ui (beta) を最新版へ追随する

Releases · mui-org/material-ui

オフラインでも利用できるようにする

• Pre-cache by service worker
• Support Google Analytics

iOS Safariで前発/次発ボタンのダブルタップが動作しない

SSIA

URLコピーの完了メッセージがdrawerメニューに重なる

drawerメニューを閉じて、すぐにsnackbarを出す挙動に変える

24時以降の発車タイマーで不必要な`00時間`表示が出る

libs/timeTableDataHandler.jsの.findNextTime内のcurrentTime時刻ずらしと、components/BoardingTimer.jsの.buildNextState内のnextTime.diff(currentTime)の不整合が原因

メニューDrawerにタブバーが重なる

Drawerがタブバーよりも前に来てほしい。

Screenshots

ホーム画面に追加バナーが出ていない可能性がある(Service Worker)

要調査。
ホーム画面に追加する方法は他にもあるのでwontfix

オフラインで利用できるようになったら通知を表示する

SSIA

時刻表テーブルの直近発車時刻の色づけがズレる

アプリ化（ホーム画面へ追加）のお知らせを表示する

SSIA

時刻表ページを刷新する

• バス停毎に表組み
• ページ内のアンカーリンク

前発/次発ボタンのダブルタップで直近/最終バスへ移動する

再実装。#8 も一緒に解決したい

新しいお知らせがあるときにバッジを付ける。見たら消える

SSIA

バスルート地図のマンション名が間違っている

誤）DEUX TROUS
正）DEUX TOURS

Migrate Now to Vercel

Packages/CLI

• package.json
  • dependencies
  • scripts
• GitHub Actions

Path

• .now directory to .vercel
• ~/.now directory to ~/.vercel
• now.json to vercel.json
• .nowignore to .vercelignore

Environment Variables

• NOW_ to VERCEL_

PaaS URL

• now.sh to vercel.app

DNS

via

• https://www.notion.so/Frequently-Asked-Questions-c726facaf7cc41c095ba49aeea90219e
• https://www.notion.so/Change-Timeline-fab7622780694753a92c0a396cea7eb6

時刻表ページのページ内アンカーIDがきれいでない

• #withStyles(TimeTable)[Home] -> Home
• #withStyles(TimeTable)[HGinza] -> HigashiGinza
• #withStyles(TimeTable)[Shimbashi] -> Shimbashi

不要なアセットファイルを削除・移動する

• 画像
• getappページ

発車タイマー更新時の動作が重い

• 他項目から計算可能な項目をstateに持たないようにする
• shouldComponentUpdateで不要なstate比較を避ける（タイマーが動かなくなる）

緊急事態宣言に伴う運休に対応する

• タイマーボードに「臨時運休中」
• お知らせ欄に「4/8(水)より東京都緊急事態措置の影響により一時運休中。再開時期は未定で、管理組合が行政見解を待ち別途告知予定」

運行日以外または運行時間外にビルドされた場合、運行時間中の表示が崩れる

SSG -> SSR. Detects current time on server-side, and pass time prop to client-side.

Support for weekend and holiday timetables

Requirements

• Appearance：タイマー・時刻表部分は「平日 or 土日祝」を容易に見分けられる
• Switching：時刻表は全体を「平日」と「土日祝」を切り替えられる (Tab)

Specifications

• 曜日・祝日判定：Containerに入れる
  • isInactiveDays, inactiveDay, isClosedDay の変数名･関数名を土日祝が休業前提ではない文言へ変更
  • nextState.outOfService = null となる条件から inactiveDay を削除
• 時刻表データ：バス停データの中に平日・土日祝の両方を格納
  • Timetable classにデータと平日・土日祝を見分けられる property を持つ

Architecture

• hooks
  • useClock：現時刻を刻む
  • useOpsDay：平日・土日祝を切り替えられる
  • useTimetable：時刻表を保持する

Clean Architectureにならう

• presenter
  • Timetableの配列をバス停毎にグループ化
  • 時刻の分と秒は0パディング

Automatically generate sitemap.xml

https://vercel.com/blog/10-next-js-tips-you-might-not-know#tip-4-hooking-into-the-build-process

タイマーから時刻表ページへリンクする

Depends on #13

• バス停名やタイマーの時刻部分タップ
• 遷移したらタップした各バス停の時刻表アンカーへ移動

緊急事態宣言の解除にともなうバス運行再開に対応する

5/27 6時から運行再開