gocodebox / gocodebox/lifterlms
create email notification tests for existing notifications
- 主要言語
- 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