gocodebox / gocodebox/lifterlms

create email notification tests for existing notifications

オープン
#653 コメント 3 件 リアクション 0 件 担当者 0 名 GitHub で見る
good first issue hacktoberfest help wanted
主要言語
PHP
スター
212
フォーク
140
平均マージ
2日 14時間
マージ済み PR(30日)
19

説明

- [ ] Course Complete
- [ ] Course Track Complete
- [ ] Enrollment
- [ ] Gateway: Manual - Payment Due
- [ ] Lesson Complete
- [ ] Payment Retry Scheduled
- [x] Purchase Receipt
- [x] Quiz Failed - has pr #676
- [x] Quiz Passed - has pr #672
- [ ] Section Complete
- [ ] Student Welcome
- [ ] Subscription Cancellation Notice

Notification controllers should implement new`get_test_settings()` method: https://github.com/gocodebox/lifterlms/blob/master/includes/abstracts/llms.abstract.notification.controller.php#L319-L328

As well as new `send_test()` method: https://github.com/gocodebox/lifterlms/blob/master/includes/abstracts/llms.abstract.notification.controller.php#L445-L455

Existing controllers should implement a method which queries existing actions which triggered the notification in order to provide a list of examples that can be used to send a test email.

This means that there must be sample data on the site which can be used to send the test. EG: a course completion test cannot be sent if there haven't been any course completions.

Example implementation example to come

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

includes/abstracts/llms.abstract.notification.controller.php から始めて get_test_settings() と send_test() を確認し、その後、チェックリストに記載されている既存の通知コントローラーと比較します。未チェックの通知のカバレッジを追加し、それぞれが既存のトリガーアクションから例を取得できるようにします。一覧にある通知を利用可能なサンプルデータでテストできれば完了です。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
php, wordpress
領域
backend, testing
issue の種類
機能追加
難易度
4/5
見積もり時間
3〜5日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
35/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。