-
Notifications
You must be signed in to change notification settings - Fork 9
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
Refactor JS code #732
Refactor JS code #732
Conversation
03ccfda
to
c44af89
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ありがとうございます。素晴らしいです。1点だけ、質問をコメントしましたので回答いただけますか?
app/assets/javascripts/files.js
Outdated
}, | ||
pageLength: 100 | ||
}); | ||
$(() => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
こちらは $( ()=>{... } )
としていますが、他と同様に即時実行関数 ( ()=>{...} )()
として書いていないのは何か意図があるのでしょうか?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ご指摘ありがとうございます。私が書き違えていました。ここも他と同様に関数呼び出しのタイミングによって未定義エラーにならないように即時実行関数を使うことを意図していますので修正します。
6cdd4c4
to
3c75fd0
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ありがとうございます
refs: #711
対応内容