takahashimasahiro / kintai_app Goto Github PK

View Code? Open in Web Editor NEW

1.0 1.0 2.0 107.15 MB

Ruby 76.25% JavaScript 4.40% CSS 2.59% HTML 16.75%

kintai_app's People

Contributors

Stargazers

Watchers

Forkers

matsudakrh szkhrnb

kintai_app's Issues

review17

kintai_app/db/migrate/20181002041408_create_users.rb

Line 4 in a0f9164

t.string :user_id

usersテーブルに
user_idを持つ意味は？何が用途か？

railsは、idベースなので不要な気がする。

railsの設計**はサロゲートキーです。
https://qiita.com/tmasu/items/1b8702d8a121cc10aefb

review19

kintai_app/app/controllers/users_controller.rb

Line 9 in a0f9164

@dateofweek = ["日","月","火","水","木","金","土"]

これがdefault standardなので使用するようにしましょう
https://github.com/svenfuchs/rails-i18n/blob/master/rails/locale/ja.yml

review15

kintai_app/app/controllers/tops_controller.rb

Line 21 in a0f9164

@email = params[:email]

form_helperを使っているのでformの復元を意識しないで良いはずです。違ったらごめんなさい。
railに乗りましょう。

https://railsguides.jp/action_view_overview.html#formhelper

      @email = params[:email]
      @password = params[:password]

review13

kintai_app/app/controllers/application_controller.rb

Line 11 in a0f9164

if @current_user == nil

set_current_user
を
current_user
として

他処理ではインスタンス変数を直参照しないで、current_userを参照するようにしましょう
@current_userはグローバル変数のようなものなので、スコープが限定されてgoodです。

全登録ユーザーの月別状況が確認できるような画面orCSVファイルが欲しい。

review10

kintai_app/app/controllers/tops_controller.rb

Line 10 in a0f9164

無駄な空行は不要

出勤データがない場合はステータスを空白、もしくは、未出勤にする。

時間が入力された場合はステータスを出勤にする。
休日の場合も同様に行う。ただし休日出勤の場合は代休が取れるように

slackbotと連携できるようにする

チャンネル作ってテストする。
API controllerを作る

review11

kintai_app/app/controllers/application_controller.rb

Line 13 in a0f9164

redirect_to('/')

url_helperを使いましょう（他も同様）

https://railsguides.jp/routing.html#%E3%83%91%E3%82%B9%E3%81%A8url%E7%94%A8%E3%83%98%E3%83%AB%E3%83%91%E3%83%BC

root_path

review7

kintai_app/db/migrate/20181002041408_create_users.rb

Line 3 in a0f9164

create_table :users do |t|

:commentオプションをつけましょう

heroku側の時間入力がおかしい

出勤押下時に現在時刻＋９時間で入力され、退勤時間が現在時刻-6時間で入力される。
退勤押下時に現在時刻＋９時間で入力され、出勤時間が現在時刻-6時間で入力される。

休暇申請するときと承認、却下するときにメモが書けるようにする。

申請者の申請理由と認証者の承認or却下理由の書けるカラムを追加
それぞれのroleは自分の申請の申請理由と承認理由が表示されるように申請関係の画面を変更。
ただし申請理由は申請者のみ編集できる。承認or却下理由はownerのみ編集できる。
それぞれの理由をどこで入力させるべきか？
1. 状態を休暇申請にしたときにモーダル表示
2.休暇確認画面で編集用カラムを追加

role以外のユーザーも他ユーザーの申請理由を閲覧できるようにするべき(出勤者リスト画面に表示できると良さそう)

review3

kintai_app/config/database.yml

Line 19 in a0f9164

encoding: unicode

rubyはutf-8です。

review9

kintai_app/app/controllers/application_controller.rb

Line 4 in a0f9164

def set_current_user

railsだと set_ を付けない慣習があります。
current_user

https://railstutorial.jp/chapters/basic_login?version=5.1#sec-current_user

review23

kintai_app/app/views/tops/top.html.erb

Line 10 in a0f9164

form_helperを使い倒してください

https://railsguides.jp/action_view_overview.html#password-field

review2

https://github.com/takahashimasahiro/kintai_app/blob/master/README.md
内容を書きましょう
概要とか
rails s できるまでの手順

が記載してあるとありがたいです。

review26

kintai_app/app/models/user.rb

Line 2 in a0f9164

validates :name, {presence: true}

日本語でエラーメッセージ出るようにしてください。
i18nを使用すること

review18

kintai_app/app/controllers/tops_controller.rb

Line 6 in a0f9164

session[:user_id] = nil

これをする意味は？

review22

kintai_app/app/views/tops/top.html.erb

Line 1 in a0f9164

<html>

railsのlayoutについて調べてください。htmlタグが二重になっています。

https://railstutorial.jp/chapters/filling_in_the_layout?version=4.2#cha-filling_in_the_layout

  | <!DOCTYPE html>
-- | --
  | <html>
  | <head>
  | <title>Kintai</title>
  | <meta name="csrf-param" content="authenticity_token" />
  | <meta name="csrf-token" content="6+CPOilyQ3oGyYsRG60fIk/8Twq60rueXyvXZyNyT1owbuA69W5aH/4MOlVIuuO6mH0PgtppE4QGRD94lAd1Yw==" />
  |  
  | <link rel="stylesheet" media="screen" href="/packs/application-745a1ca3d5f8eb84a4396994591ab7fd.css" />
  | <link rel="stylesheet" media="screen" href="/packs/tops-82cfc99e066de857bd4da8b9444179b3.css" />
  | <script src="/packs/application-c0233acbd733caeafd45.js"></script>
  | <script src="/packs/tops-c7e9550d0c08a0728bf8.js"></script>
  | </head>
  | <body>
  | <div class="wrapper-logout">
  | <html>

review20

kintai_app/config/application.rb

Line 21 in a0f9164

class Application < Rails::Application

timezoneについて気にするようにしてください。defaultだとutcになった覚えがある

config.time_zone = 'Tokyo'

review6

kintai_app/config/routes.rb

Line 3 in a0f9164

post 'login' => 'tops#login'

セッション管理はSessionsControllerにするのが良いです。
コントローラー名とやっていることを同じにするのが良いです。

https://railstutorial.jp/chapters/basic_login?version=5.1#sec-sessions_controller

review1

kintai_app/.gitignore

Line 41 in a0f9164

vendor/bundler

これだとbundle配下が管理対象となってしまう
パスの書き方が大丈夫かを全体的に見直してください

review12

kintai_app/app/controllers/application_controller.rb

Line 12 in a0f9164

flash[:notice] = 'ログインが必要です'

はじめなのでi18nを厳密に使ってみてください。（実務だと時間がなくてできないが、正道は使う）

https://railsguides.jp/i18n.html

review27

kintai_app/spec/controllers/tops_controller_spec.rb

Line 3 in a0f9164

RSpec.describe TopsController, type: :controller do

http://www.betterspecs.org/jp/

describeとcontextを使い分ける
などを覚えてください

review21

kintai_app/app/controllers/users_controller.rb

Line 10 in a0f9164

@today = Date.today

utc表示にならないかチェックする
https://qiita.com/jnchito/items/cae89ee43c30f5d6fa2c

review16

kintai_app/app/controllers/application_controller.rb

Line 17 in a0f9164

def fobid_current_user

何もメッセージ出さずにログイン後のトップ画面にredirectするのではダメですか？

return Post.where(user_id: self.id)

rubyはreturnあまり使わない文化らしいです。

review24

kintai_app/app/helpers/top_helper.rb

Line 1 in a0f9164

module TopHelper

いらないファイルは消す

review4

kintai_app/config/database.yml

Line 27 in a0f9164

adapter: postgresql

共通事項はdefaultのみにしてください

  adapter: postgresql
  encoding: unicode

before_action :authenticate_current_user, {only: [:logout]}

logoutするのに敢えて認証のチェックする理由は？

review8

kintai_app/app/controllers/application_controller.rb

Line 6 in a0f9164

@current_user = User.find_by(user_id: session[:user_id])

キャッシュしましょう
@current_user ||= User.find_by(user_id: session[:user_id])

https://railstutorial.jp/chapters/basic_login?version=5.1#cha-basic_login

review5

kintai_app/config/routes.rb

Line 2 in a0f9164

get 'logout' => 'tops#logout'

はじめなので基本に忠実にRESTfulにしましょう
railsはrailに乗るのが良いらしいです。

https://railsguides.jp/routing.html#%E3%83%AA%E3%82%BD%E3%83%BC%E3%82%B9%E3%83%99%E3%83%BC%E3%82%B9%E3%81%AE%E3%83%AB%E3%83%BC%E3%83%86%E3%82%A3%E3%83%B3%E3%82%B0-rails%E3%81%AE%E3%83%87%E3%83%95%E3%82%A9%E3%83%AB%E3%83%88

takahashimasahiro / kintai_app Goto Github PK

kintai_app's People

Contributors

Stargazers

Watchers

Forkers

kintai_app's Issues

Recommend Projects

Recommend Topics

Recommend Org

Jobs