Skip to content

Conversation

@debdutdeb
Copy link
Member

Proposed changes (including videos or screenshots)

Issue(s)

Steps to test or reproduce

Further comments

@debdutdeb debdutdeb marked this pull request as ready for review April 22, 2022 18:27
ggazzo
ggazzo previously requested changes Apr 26, 2022
@debdutdeb debdutdeb requested a review from a team as a code owner May 6, 2022 14:37
@debdutdeb debdutdeb requested a review from ggazzo May 6, 2022 14:38
@tassoevan tassoevan dismissed stale reviews from gabriellsh and ggazzo May 10, 2022 10:34

Resolved

@tassoevan tassoevan changed the title Chore: Convert to ts: UserStatusMenu Chore: Convert UserStatusMenu to TS May 10, 2022
@tassoevan tassoevan merged commit e1588e0 into develop May 10, 2022
@tassoevan tassoevan deleted the ts-rewrite/UserStatusMenu branch May 10, 2022 10:35
gabriellsh added a commit that referenced this pull request May 12, 2022
…lists

* 'develop' of github.com:RocketChat/Rocket.Chat:
  Chore: Move admin sidebarItems registration to the main file (#25442)
  [FIX] Sanitize customUserStatus and fix infinite loop (#25449)
  [IMPROVE] Fix multiple bugs with Matrix bridge (#25318)
  Chore: Convert `UserStatusMenu` to TS (#25265)
  Chore: Chore add validation option to rest endpoints (#25443)
  Chore: Add channel endpoints (rest-typings) (#25279)
  Update Codeowners
  Chore: Dedicated package for UI contexts (#25432)
@d-gubert d-gubert mentioned this pull request May 31, 2022
debdutdeb added a commit that referenced this pull request Jun 5, 2022
… dropdown menu (#24796)

* Invisible to Offline

* accomodate #25265  (Convert to tsx)+

* Rename UserStatusMenu.js to UserStatusMenu.tsx

* Use correct file path

Co-authored-by: Debdut Chakraborty <debdut.chakraborty@rocket.chat>
Co-authored-by: dougfabris <devfabris@gmail.com>
@murtaza98 murtaza98 mentioned this pull request Jul 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants