Skip to content

[FIX] Rooms list sorting by activity multiple re-renders and case sensitive sorting alphabetically#9959

Merged
sampaiodiego merged 29 commits intodevelopfrom
improve
Jun 10, 2018
Merged

[FIX] Rooms list sorting by activity multiple re-renders and case sensitive sorting alphabetically#9959
sampaiodiego merged 29 commits intodevelopfrom
improve

Conversation

@ggazzo
Copy link
Member

@ggazzo ggazzo commented Mar 1, 2018

@RocketChat/core

Closes #10004
Closes #10415
Closes #9253
Closes #9295
Closes #5341

@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-9959 March 1, 2018 18:00 Inactive
@ggazzo ggazzo temporarily deployed to rocket-chat-pr-9959 March 1, 2018 18:06 Inactive
@RocketChat RocketChat deleted a comment Mar 1, 2018
@ggazzo ggazzo temporarily deployed to rocket-chat-pr-9959 March 15, 2018 21:30 Inactive
@ggazzo ggazzo temporarily deployed to rocket-chat-pr-9959 March 15, 2018 21:40 Inactive
@ggazzo ggazzo temporarily deployed to rocket-chat-pr-9959 March 23, 2018 16:02 Inactive
@ggazzo ggazzo requested review from karlprieb, rodrigok and sampaiodiego and removed request for karlprieb March 23, 2018 16:46
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is not working as expected.. it is showing offline for all users

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

are you sure?
image

Copy link
Member

@sampaiodiego sampaiodiego Mar 27, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

did you test with other users or just yours?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sure
image

}
});

RocketChat.callbacks.add('enter-room', (sub) => {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

my channels did not have active indicators while testing

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed :)

@ggazzo ggazzo temporarily deployed to rocket-chat-pr-9959 March 27, 2018 00:16 Inactive
@ggazzo ggazzo temporarily deployed to rocket-chat-pr-9959 March 27, 2018 00:17 Inactive
@ggazzo ggazzo temporarily deployed to rocket-chat-pr-9959 March 28, 2018 17:24 Inactive
@ggazzo ggazzo temporarily deployed to rocket-chat-pr-9959 May 26, 2018 02:17 Inactive
@ggazzo ggazzo temporarily deployed to rocket-chat-pr-9959 May 26, 2018 02:41 Inactive
@ggazzo
Copy link
Member Author

ggazzo commented Jun 6, 2018

image

@ggazzo ggazzo temporarily deployed to rocket-chat-pr-9959 June 7, 2018 18:36 Inactive
@ggazzo ggazzo temporarily deployed to rocket-chat-pr-9959 June 7, 2018 18:36 Inactive
@sampaiodiego sampaiodiego temporarily deployed to rocket-chat-pr-9959 June 7, 2018 20:59 Inactive
@sampaiodiego sampaiodiego temporarily deployed to rocket-chat-pr-9959 June 8, 2018 11:46 Inactive
@sampaiodiego sampaiodiego temporarily deployed to rocket-chat-pr-9959 June 8, 2018 13:33 Inactive
@sampaiodiego sampaiodiego temporarily deployed to rocket-chat-pr-9959 June 8, 2018 16:37 Inactive
@sampaiodiego sampaiodiego temporarily deployed to rocket-chat-pr-9959 June 8, 2018 19:37 Inactive
@engelgabriel engelgabriel modified the milestones: 0.65.0, 0.66.0 Jun 8, 2018
@sampaiodiego sampaiodiego changed the title [FIX] improves to use a cursor to render roomlist [FIX] Rooms list sorting by activity multiple re-renders and case sensitive sorting alphabetically Jun 10, 2018
@sampaiodiego sampaiodiego merged commit 725d3e0 into develop Jun 10, 2018
@sampaiodiego sampaiodiego deleted the improve branch June 10, 2018 21:49
@ggazzo
Copy link
Member Author

ggazzo commented Jun 10, 2018

\o/

trongthanh added a commit to goalifyplus/Goalify.Chat that referenced this pull request Jun 11, 2018
* develop: (215 commits)
  [FIX] Rooms list sorting by activity multiple re-renders and case sensitive sorting alphabetically (RocketChat#9959)
  [FIX] Notification not working for group mentions and not respecting ignored users (RocketChat#11024)
  fixed searchbar icon and text overlap (RocketChat#10294)
  [FIX] Link previews not being removed from messages after removed on editing (RocketChat#11063)
  avoid send presence without loggin
  Fix: Exception in metrics generation
  Update release template
  Update issue templates
  Update pt-BR.i18n.json
  LingoHub Update 🚀
  update import capnp
  update build for Sandstorm, add dependence capnp.
  LingoHub Update 🚀
  Fixing HTML on translation files
  Fixing HTML on translation files
  LingoHub Update 🚀
  Fixing HTML on translation files
  Fixing HTML on translation files
  Fixing HTML on translation files
  LingoHub Update 🚀
  ...

# Conflicts:
#	.meteor/packages
#	README.md
#	client/startup/startup.js
#	package-lock.json
#	packages/rocketchat-error-handler/server/lib/RocketChat.ErrorHandler.js
#	packages/rocketchat-i18n/i18n/en.i18n.json
#	packages/rocketchat-i18n/i18n/vi-VN.i18n.json
#	packages/rocketchat-livechat/config.js
#	server/startup/initialData.js
trongthanh added a commit to goalifyplus/Goalify.Chat that referenced this pull request Jun 11, 2018
* goalify: (217 commits)
  Add more scripts to reduce mannual interaction during server init
  [FIX] Rooms list sorting by activity multiple re-renders and case sensitive sorting alphabetically (RocketChat#9959)
  [FIX] Notification not working for group mentions and not respecting ignored users (RocketChat#11024)
  fixed searchbar icon and text overlap (RocketChat#10294)
  [FIX] Link previews not being removed from messages after removed on editing (RocketChat#11063)
  avoid send presence without loggin
  Fix: Exception in metrics generation
  Update release template
  Update issue templates
  Update pt-BR.i18n.json
  LingoHub Update 🚀
  update import capnp
  update build for Sandstorm, add dependence capnp.
  LingoHub Update 🚀
  Fixing HTML on translation files
  Fixing HTML on translation files
  LingoHub Update 🚀
  Fixing HTML on translation files
  Fixing HTML on translation files
  Fixing HTML on translation files
  ...
@rodrigok rodrigok mentioned this pull request Jun 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: ui Touches the code on client side

Projects

None yet

5 participants

Comments