This repository was archived by the owner on Nov 5, 2025. It is now read-only.
Chore: Add description value to IRoom interface#380
Merged
Conversation
|
|
lucassartor
changed the title
lucassartor
changed the title
Codecov Report
@@ Coverage Diff @@
## alpha #380 +/- ##
==========================================
+ Coverage 49.69% 50.24% +0.54%
==========================================
Files 100 108 +8
Lines 3151 3324 +173
Branches 463 464 +1
==========================================
+ Hits 1566 1670 +104
- Misses 1585 1654 +69
Continue to review full report at Codecov.
|
d-gubert
approved these changes
Mar 19, 2021
thassiov
pushed a commit
that referenced
this pull request
Apr 7, 2021
d-gubert
added a commit
that referenced
this pull request
Jun 1, 2021
* Start bridge refactor * Add description value to IRoom interface (#380) * Remove old permission checkers * Add new bridge classes * Remove references to `proxy` code from the permission manager * Adjust types at bridges * Change the method for checking permissions and notify the errors * Rename exposed methods to use the permission methods * Add internal bridge interfaces * Cast methods from app manager related * Adjust spies on unit tests * Include the `delete room` method in the refactor * Update indentation based on reviewer's feedback * Switch from IUserBridge interface to UserBridge abstract class * Switch from IRoomBridge interface to RoomBridge abstract class * Switch from IPersistenceBridge interface to PersistenceBridge abstract class * Switch from ILivechatBridge interface to LivechatBridge abstract class * Switch from IMessageBridge interface to MessageBridge abstract class * Switch from IUploadBridge interface to UploadBridge abstract class * Switch from IHttpBridge interface to HttpBridge abstract class * Adjust argument type from delete room method * Switch from IEnvVarBridge interface to EnvVarBridge abstract class * Switch from IAppApiBridge interface to ApiBridge abstract class * Switch from IAppSchedulerBridge interface to SchedulerBridge abstract class * Switch from IAppCommandBridge interface to CommandBridge abstract class * Switch from IServerSettingBridge interface to ServerSettingBridge abstract class * Switch from IUiInteractionBridge interface to UiInteractionBridge abstract class * Switch from IAppDetailChangesBridge interface to AppDetailChangesBridge abstract class * Switch from IAppActivationBridge interface to AppActivationBridge abstract class * Rename SettingBridge to ServerSettingBridge * Adjustments based on linter hints * Rename permission check methods prefix from 'check' to 'has' * Adjust tests to work with bridges' abstract classes * Adjustments based on linter hints * Fix problems of lack of permissions in test suites Co-authored-by: thassiov <tvmcarvalho@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What? ⛵
Added the
descriptionvalue to aIRoominterface. Thedescriptionvalue refers to a room topic.Why? 🤔
To allow developers to get the
descriptioninformation from a room.Links 🌎
Fixes #203
PS 👀