Skip to content

Conversation

@tapiarafael
Copy link
Contributor

@tapiarafael tapiarafael commented Aug 5, 2022

Proposed changes (including videos or screenshots)

Fix the bug where normal users cannot open an app contextual bar.
The request made by the contextual bar to get the app information, which was for admin only, was removed since the response was not being used.

Issue(s)

Steps to test or reproduce

Further comments

@tapiarafael tapiarafael requested a review from a team as a code owner August 5, 2022 18:43
@codecov
Copy link

codecov bot commented Aug 5, 2022

Codecov Report

Merging #26495 (20c2e17) into develop (15b0931) will increase coverage by 0.12%.
The diff coverage is 50.00%.

❗ Current head 20c2e17 differs from pull request most recent head aa85eaa. Consider uploading reports for the commit aa85eaa to get more accurate results

Impacted file tree graph

@@             Coverage Diff             @@
##           develop   #26495      +/-   ##
===========================================
+ Coverage    38.04%   38.16%   +0.12%     
===========================================
  Files          759      759              
  Lines        18831    18829       -2     
  Branches      1459     1459              
===========================================
+ Hits          7164     7186      +22     
+ Misses       11445    11421      -24     
  Partials       222      222              
Flag Coverage Δ
e2e 38.16% <50.00%> (+0.12%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

@tapiarafael tapiarafael changed the title [FIX] Allow role user to open contextual bar [FIX] Allow user role to open contextual bar Aug 8, 2022
@tapiarafael tapiarafael added this to the 5.0.3 milestone Aug 10, 2022
@tapiarafael tapiarafael requested a review from a team as a code owner August 10, 2022 21:38
@dougfabris dougfabris changed the title [FIX] Allow user role to open contextual bar [FIX] Allow normal user to open apps contextual bar Aug 10, 2022
@sampaiodiego sampaiodiego removed this from the 5.0.3 milestone Aug 10, 2022
@ggazzo ggazzo added stat: ready to merge PR tested and approved waiting for merge and removed stat: needs QA labels Aug 11, 2022
@kodiakhq kodiakhq bot merged commit fcf2864 into develop Aug 11, 2022
@kodiakhq kodiakhq bot deleted the fix/apps-contextual-bar branch August 11, 2022 17:17
gabriellsh added a commit that referenced this pull request Aug 12, 2022
…hreads

* 'develop' of github.com:RocketChat/Rocket.Chat: (34 commits)
  Chore: update fuselage rounded edition  (#26540)
  [FIX] LDAP fails to sync teams when the user DN has escaped characters. (#26535)
  [FIX] Allow normal user to open apps contextual bar (#26495)
  Chore: restrict `.only` (#26537)
  [FIX] Endpoints not working when using "Use Real Name" setting (#26530)
  [NEW] `Home` page (#25734)
  Chore: ModalFooterControllers adoption (#26445)
  Chore: create roomNameExists endpoint (#26386)
  Chore: Improve test for livechat  (#26527)
  Chore: Fix UiKit dependency issue for Livechat (#26534)
  [FIX] Too many REST API requests (#26330)
  Chore: Mocha handling multiple React instances (#26513)
  Chore: Convert `LivechatCustomField` model to raw model (#26446)
  i18n: Language update from LingoHub 🤖 on 2022-08-08Z (#26508)
  [FIX] Open team after room not found page (#26264)
  Chore: Refactor ReportMessage Modal to React Component (#26478)
  Chore: Fix lint issues (#26531)
  [FIX] Don't wrap wrap up notes (#26375)
  [IMPROVE] OTR refactoring (#24757)
  [FIX] Prevent VoIP issues during disconnection when network failed (#26321)
  ...
@murtaza98 murtaza98 mentioned this pull request Sep 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stat: ready to merge PR tested and approved waiting for merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants