thinkingreed-inc / oss-calendar

OSS Calendarは企業で使うことを想定したオープンソースのカレンダーアプリです。
https://oss-calendar.com/
30 stars 12 forks source link

[PHP8.1への対応]各修正のまとめ #36

Closed kkouta closed 1 year ago

kkouta commented 2 years ago

関連Issue / Related Issue

不具合・対応の内容 / Bug

  1. マイグレーションファイルのdownに同じカラムが記述されている。
  2. 必須項目のラベルに「*」を加える対応。
  3. OSSカレンダーのPHP8.1への対応。
  4. バックアップに伴う、レイアウトの修正とパスワードリセットへの対応。
  5. 各マスタ、ユーザー管理などのリストが表示順で並ぶように修正。
  6. 部署管理の部署階層設定がうまく動かないため、一時的に非表示にする対応。
  7. 表示順項目に全角文字を入れると0になる件を修正。 -半角の数字のみ入力できるように修正。
  8. READMEの修正。
  9. 予定作成画面にて、デフォルトで部署に「全て」が選択されるようにする対応。
  10. グループ週、グループ日を表示する際にID順ではなくユーザー名順になるように修正。 -並び順の部分をクリックすることでID順やユーザー名の降順にすることも可能。
  11. ユーザー管理画面で管理者ユーザーが削除されないようにする修正。 -一番最初に生成される「admin」ユーザーの削除と無効ボタンを使えないようにした。
  12. ユーザー名項目が空の場合にユーザー検索が正常に動作しない問題を修正。
  13. 検索時にユーザー名を検索対象にするよう対応。
  14. ユーザー登録時に、削除されたユーザーと同じユーザー名、メールアドレスのユーザーが作成できない問題を修正。
  15. 削除した予定タイプを使用した予定が登録されていると、カレンダーを開いたときにエラーになる問題を修正。 -削除した予定タイプを使用していた予定は、その削除された予定タイプを持ったままになっている。
  16. 一般ユーザーにてカレンダーを表示する際、アラートメッセージが表示される問題を修正。
  17. 予定を登録した時に公開設定が選択されない問題を修正。 -クイック作成から予定を作成した時など、何も選択されなかったときは公開設定に一般公開を設定する。
  18. 繰り返し予定の終了日が翌月カレンダーに表示される範囲に入っていないと表示されない問題を修正。
  19. 予定削除時に繰り返し設定のdisable設定を解除するように修正。
  20. 新規登録画面にて、長い名称もポップアップで確認できるように修正。 -新規登録のウィンドウをユーザーのwebウィンドウの大きさに追わせて大きく表示するように修正。それでも画面に入りきらない長い名称が出てきたときは、カーソルを合わせることでポップアップですべて表示するように修正。
  21. スマホでカレンダーをクリックしたときに、デフォルトで開始日がクリックした日付になるように修正。 -修正前はスマホで操作するときだけ、どの日をクリックしてもデフォルトで開始日が今日になってしまっていた。

スクリーンショット・修正の内容 / Screenshot

番号は 不具合・対応の内容 の番号に対応している。

  1. 必須項目に「*」を加えるように修正 image

  2. 修正前 image 修正後 image

  3. 全角数字を入力した時 image

  4. 修正前は「公開」の設定に何も選択されなかった。 image

  5. image

  6. 修正前は削除したユーザー名と同じユーザー名で登録するとエラーが発生していた。 image

  7. 詳細を開くと削除された予定タイプがそのまま表示されるようになっている。 image

  8. 下記のようなエラーが発生しないように修正。 image

  9. image

  10. 予定削除時に下記のように繰り返し設定が選択できない場合があった。この問題を修正。 image

  11. image

  12. スマートフォンで操作するときも開始日にクリックした日付が入るように修正された。 image

チェックリスト / Check List

備考 / Remarks

Remicck commented 1 year ago

https://github.com/thinkingreed-inc/oss-calendar/pull/42 Conflictを解消したPRを作成したためClose