coveralls
commented
1 year ago Pull Request Test Coverage Report for Build 4308554443
- 0 of 0 changed or added relevant lines in 0 files are covered.
- 11 unchanged lines in 6 files lost coverage.
- Overall coverage decreased (-0.03%) to 84.935%
| Files with Coverage Reduction | New Missed Lines | % | ||
|---|---|---|---|---|
| src/admin/versions.js | 1 | 83.33% | ||
| src/messaging/notifications.js | 1 | 94.83% | ||
| src/controllers/admin/dashboard.js | 2 | 85.26% | ||
| src/database/mongo/sorted.js | 2 | 92.7% | ||
| src/middleware/admin.js | 2 | 71.54% | ||
| src/user/approval.js | 3 | 87.39% | ||
| <!-- | Total: | 11 | --> |
| Totals | |
|---|---|
| Change from base Build 4301270521: | -0.03% |
| Covered Lines: | 22580 |
| Relevant Lines: | 25344 |
I added a Pin option in the topic reply's tools menu. After you click on the three dots in your reply, the tools menu shows up, and the Pin option appears under the Edit option. When you click on the Pin option, a pop-up window for the administrator to set the end date of pinning of the reply appears.
I modified the themes/nodebb-theme-persona/templates/partials/topic/post-menu-list.tpl file to include a Pin option. I refered to the topic-pin function in themes/nodebb-theme-persona/templates/partials/topic/topic-menu-list.tpl to include a pop-up window for the administrator to set the end date of pinning of the reply whenever the Pin option is selected.
This is the front-end implementation of issue #4. Needs further back-end support.