[NEW] Ability for apps-engine to archive or unarchive rooms
Proposed changes (including videos or screenshots)
Issue(s)
Steps to test or reproduce
Further comments
apps pr: https://github.com/RocketChat/Rocket.Chat.Apps-engine/pull/535 example app: https://gist.github.com/debdutdeb/4fcb5fa6e4082f361aa9022572f9c107
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 41.22%. Comparing base (
dc8b46d) to head (ea58d2c). Report is 2969 commits behind head on develop.
:exclamation: Current head ea58d2c differs from pull request most recent head 4ca1f73. Consider uploading reports for the commit 4ca1f73 to get more accurate results
Additional details and impacted files
@@ Coverage Diff @@
## develop #26317 +/- ##
===========================================
- Coverage 41.23% 41.22% -0.02%
===========================================
Files 839 839
Lines 18254 18254
Branches 2065 2065
===========================================
- Hits 7527 7525 -2
- Misses 10439 10443 +4
+ Partials 288 286 -2
| Flag | Coverage Δ | |
|---|---|---|
| e2e | 41.22% <0.00%> (-0.02%) |
:arrow_down: |
Flags with carried forward coverage won't be shown. Click here to find out more.
This pull request introduces 1 alert when merging 948aadd95a3145048c19f11bdc3c3031cff38928 into ca29977236a40014c39100247722599ad5177e2b - view on LGTM.com
new alerts:
- 1 for Syntax error
Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine :gear: that powers LGTM.com. For more information, please check out our post on the GitHub blog.
This pull request introduces 1 alert when merging c54016cd46b46529d74a65661e768099f0fb785d into ea58d2cc8547a330f493fee0d6bc0b86d6271a2b - view on LGTM.com
new alerts:
- 1 for Syntax error
Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine :gear: that powers LGTM.com. For more information, please check out our post on the GitHub blog.
This pull request introduces 1 alert when merging 4ca1f73666ad7c6d591f61bc2e63422a61a4354d into dc8b46df01cb55f7f589ca5621c56df244c13f83 - view on LGTM.com
new alerts:
- 1 for Syntax error
Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine :gear: that powers LGTM.com. For more information, please check out our post on the GitHub blog.
Looks like this PR is not ready to merge, because of the following issues:
- This PR has conflicts, please resolve them before merging
- This PR is missing the 'stat: QA assured' label
- This PR is not mergeable
- This PR is missing the required milestone or project
- This PR has an invalid title
Please fix the issues and try again
If you have any trouble, please check the PR guidelines
Looks like this PR is not ready to merge, because of the following issues:
- This PR has conflicts, please resolve them before merging
- This PR is missing the 'stat: QA assured' label
- This PR is not mergeable
- This PR is missing the required milestone or project
- This PR has an invalid title
Please fix the issues and try again
If you have any trouble, please check the PR guidelines