Skip to content

Conversation

@EiThandarAung-pressman
Copy link
Collaborator

フィイルをPull Request出しました。ご確認よろしくお願い致します。

@pm-hiroshi pm-hiroshi changed the title File review 初級 review Sep 15, 2023
@pm-hiroshi
Copy link
Contributor

app/public/wp-content/plugins/alert-on-login/alert-on-login.php
へのレビューです。
*変更部分しかコメントできなかったので、この形でのレビューになります。

line 43の

add_action('init', array($this, 'enqueue_custom_js'), 10);
のenqueue_custom_jsは、動かないか、エラーが出ると思います。
参考ページ : https://codex.wordpress.org/Plugin_API/Action_Reference

acfのhookでfieldのnameを指定しているので、line 86、line 96 の if() がいらないです。

line142 の == は、=== のほうがいいのですよね?

line159のfunction 'custom_column_notification'について。
イさんはWordPressのデフォルトのAPIで解決していますが、
カリキュラム「Admin Columns PRO」の課題2で指示されている解決方法と違います。もう一度、問題を読んでみてください。

@EiThandarAung-pressman

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.

3 participants