[FIX] Unexpected open or close visitor info #21094
Merged
Conversation
This file contains 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
|
This pull request introduces 1 alert when merging b78c097 into 73ea7a1 - view on LGTM.com new alerts:
|
tassoevan
requested changes
Mar 12, 2021
Co-authored-by: Tasso Evangelista <[email protected]>
|
@renatobecker The milestone for this PR is 3.13. Could you include it in 3.12.x? |
tassoevan
approved these changes
Mar 16, 2021
pierre-lehnen-rc
added a commit
that referenced
this pull request
Mar 22, 2021
* add folder structure * Create ChannelAutoCompleteMultiple component * Create GenericModal with Variants * remove enum * [WIP] Create Team Modal * add pagination prop to GenericTable * create teams modal * remove ChannelAutoCompleteMultiple component * set description in team * fix redirect to room * remove braces from simple string Co-authored-by: Tasso Evangelista <[email protected]> * single coat instead bool * refactor useReactModal * Move CreateTeam module * Split TeamNameInput * Convert BaseAvatar and UserAvatar to TypeScript * Add UsersInput * Convert CreateTeamModal to TypeScript * Upgrade Fuselage * update generic modal * Fix team name validation * Update i18n strings * Add autofocus * Pass translation-check * Fix goToRoomById call * Fix JSX expression * [FIX] Unexpected open or close visitor info (#21094) * fix unexpected open or close visitor info * fix lgtm bot warning Co-authored-by: Tasso Evangelista <[email protected]> Co-authored-by: Tasso Evangelista <[email protected]> Co-authored-by: Renato Becker <[email protected]> * Type check useMethod hook * Type check useEndpoint hook * Type check useTranslation hook * [NEW] use _rocketchatLogger in Apps debugLog (#21000) * Remove redundant polyfill * [IMPROVE] Improve Apps permission modal (#21193) * improve modal design * add en and pt-BR translations * fix icon size * [NEW][APPS] Map description as a room value in Apps (#20811) * map 'description' as a room value * Update Apps-Engine version * [FIX] Wrong license seats number administration info panel (#21222) * Use active users to display on limit of users for the license on info panel * Print more info about issues with licenses on server Co-authored-by: dougfabris <[email protected]> Co-authored-by: gabriellsh <[email protected]> Co-authored-by: Tiago Evangelista Pinto <[email protected]> Co-authored-by: Gabriel Henriques <[email protected]> Co-authored-by: Tasso Evangelista <[email protected]> Co-authored-by: Renato Becker <[email protected]> Co-authored-by: meomay503 <[email protected]> Co-authored-by: Lucas Sartor Chauvin <[email protected]> Co-authored-by: Rodrigo Nascimento <[email protected]>
pierre-lehnen-rc
added a commit
that referenced
this pull request
Mar 23, 2021
* add folder structure * Rewrite Infopanels, create teaminfo * Convert buttons to actions * Create ChannelAutoCompleteMultiple component * Create GenericModal with Variants * wip * Create GenericModal with Variants * Create GenericModal with Variants * WIP DeleteTeamModal * add visual components * remove enum * [WIP] Create Team Modal * add pagination prop to GenericTable * Separate files, table logic * lint * create teams modal * remove ChannelAutoCompleteMultiple component * set description in team * fix redirect to room * remove braces from simple string Co-authored-by: Tasso Evangelista <[email protected]> * single coat instead bool * refactor useReactModal * Move CreateTeam module * Split TeamNameInput * Convert BaseAvatar and UserAvatar to TypeScript * Add UsersInput * Convert CreateTeamModal to TypeScript * Upgrade Fuselage * WIP Co-authored-by: Douglas Fabris <[email protected]> * update generic modal * Fix team name validation * Update i18n strings * Add autofocus * Pass translation-check * Fix goToRoomById call * Fix JSX expression * [FIX] Unexpected open or close visitor info (#21094) * fix unexpected open or close visitor info * fix lgtm bot warning Co-authored-by: Tasso Evangelista <[email protected]> Co-authored-by: Tasso Evangelista <[email protected]> Co-authored-by: Renato Becker <[email protected]> * Type check useMethod hook * Type check useEndpoint hook * Type check useTranslation hook * [NEW] use _rocketchatLogger in Apps debugLog (#21000) * Remove redundant polyfill * [IMPROVE] Improve Apps permission modal (#21193) * improve modal design * add en and pt-BR translations * fix icon size * [NEW][APPS] Map description as a room value in Apps (#20811) * map 'description' as a room value * Update Apps-Engine version * WIP * [FIX] Wrong license seats number administration info panel (#21222) * Use active users to display on limit of users for the license on info panel * Print more info about issues with licenses on server * Delete & Leave Modals * wip * Fix bugs and remove logs * expose teams fields to local collection * Add teams section and team icon logic * Add prototype of teams.listRoomsOfUser * fix leave room array * Add tabbar actions to team room * Review * backend integration * Review * Review * Review * Review * Review * hide renderTrackHorizontal on rc-scrollbars Co-authored-by: dougfabris <[email protected]> Co-authored-by: Gabriel Henriques <[email protected]> Co-authored-by: gabriellsh <[email protected]> Co-authored-by: Tiago Evangelista Pinto <[email protected]> Co-authored-by: Tasso Evangelista <[email protected]> Co-authored-by: Douglas Fabris <[email protected]> Co-authored-by: Renato Becker <[email protected]> Co-authored-by: meomay503 <[email protected]> Co-authored-by: Lucas Sartor Chauvin <[email protected]> Co-authored-by: Rodrigo Nascimento <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Proposed changes (including videos or screenshots)
The VisitorInfo component closes or open every time a new message was sent, this PR fix that.
Issue(s)
Steps to test or reproduce
Further comments