Skip to content

Conversation

@juliajforesti
Copy link
Contributor

@juliajforesti juliajforesti commented Apr 24, 2023

DSAR-122
DSAR-125

Proposed changes (including videos or screenshots)

  • fix colors of New messages room button:
    from:
    image
    to:
    image
    image

  • refactor JumpToMessagesBar to a reusable component for both New messages and Jump to recent message
    from:
    image
    to:
    image

Issue(s)

Steps to test or reproduce

Further comments

refactor: JumpToMessages
@codecov
Copy link

codecov bot commented Apr 25, 2023

Codecov Report

Merging #29033 (11740d9) into develop (0fbd907) will increase coverage by 8.62%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop   #29033      +/-   ##
===========================================
+ Coverage    37.48%   46.10%   +8.62%     
===========================================
  Files          677      728      +51     
  Lines        13440    14087     +647     
  Branches      2129     2193      +64     
===========================================
+ Hits          5038     6495    +1457     
+ Misses        8126     7279     -847     
- Partials       276      313      +37     
Flag Coverage Δ
e2e 46.07% <ø> (+8.59%) ⬆️

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

@juliajforesti juliajforesti marked this pull request as ready for review April 25, 2023 17:35
@juliajforesti juliajforesti requested a review from a team as a code owner April 25, 2023 17:35
@dionisio-bot dionisio-bot bot added stat: ready to merge PR tested and approved waiting for merge and removed stat: needs QA labels Apr 26, 2023
@dougfabris dougfabris added this to the 6.3.0 milestone Apr 26, 2023
@ggazzo ggazzo merged commit 90ee529 into develop Apr 26, 2023
@ggazzo ggazzo deleted the fix/jump-to-message-btn branch April 26, 2023 20:04
gabriellsh added a commit that referenced this pull request Apr 27, 2023
…ixSearch

* 'develop' of github.com:RocketChat/Rocket.Chat: (54 commits)
  feat: Add support to search for public rooms through the Matrix Network (#28330)
  chore: Improve bash script and add new detailed infos (#29066)
  chore: Remove duplicated key in turbo.json (#29057)
  chore: improve unread indicator colors (#29045)
  fix: Notification badge background (#29053)
  fix: `JumpToMessages` button style (#29033)
  refactor: Remove imports based on isomorphic logic - 2 (#29065)
  refactor: Move `cloud` and `channel settings` to typescript (#28907)
  refactor: Separate isomorphic imports on utils - 1 (#29052)
  fix: bug in close method of RoomManager occurs memory leak (#29058)
  refactor: Remove Meteor.userId from Slashcommands (#28925)
  refactor: Move callbacks to async (#29051)
  Bump version to 6.2.0-rc.4
  refactor: No reference to jQuery types (#28977)
  regression: SlackBridge settings UI crashing (#29050)
  fix: `HeaderTag` (#28961)
  chore(deps): Patch dependencies (#29024)
  regression: Fixed livechat notifications displaying [object Promise] on title (#29021)
  regression: `fetch` response (#29032)
  refactor: Remove `meteor/http` for `serverFetch` (#29038)
  ...
gabriellsh added a commit that referenced this pull request Apr 27, 2023
…externalAvatarProvider

* 'develop' of github.com:RocketChat/Rocket.Chat: (88 commits)
  feat: Add support to search for public rooms through the Matrix Network (#28330)
  chore: Improve bash script and add new detailed infos (#29066)
  chore: Remove duplicated key in turbo.json (#29057)
  chore: improve unread indicator colors (#29045)
  fix: Notification badge background (#29053)
  fix: `JumpToMessages` button style (#29033)
  refactor: Remove imports based on isomorphic logic - 2 (#29065)
  refactor: Move `cloud` and `channel settings` to typescript (#28907)
  refactor: Separate isomorphic imports on utils - 1 (#29052)
  fix: bug in close method of RoomManager occurs memory leak (#29058)
  refactor: Remove Meteor.userId from Slashcommands (#28925)
  refactor: Move callbacks to async (#29051)
  Bump version to 6.2.0-rc.4
  refactor: No reference to jQuery types (#28977)
  regression: SlackBridge settings UI crashing (#29050)
  fix: `HeaderTag` (#28961)
  chore(deps): Patch dependencies (#29024)
  regression: Fixed livechat notifications displaying [object Promise] on title (#29021)
  regression: `fetch` response (#29032)
  refactor: Remove `meteor/http` for `serverFetch` (#29038)
  ...
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