[IMPROVE] Make debug logs of Apps configurable via Log_Level setting in the Admin panel #21000
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
cuonghuunguyen
changed the title
Log_Level setting in the Admin panel
shiqimei
approved these changes
Mar 6, 2021
shiqimei
approved these changes
Mar 6, 2021
|
Hi, do you have any update on this PR :D |
gabriellsh
added a commit
that referenced
this pull request
Mar 20, 2021
…ams/info * 'new/teams' of github.com:RocketChat/Rocket.Chat: (53 commits) [NEW][APPS] Map description as a room value in Apps (#20811) [IMPROVE] Improve Apps permission modal (#21193) Remove redundant polyfill [NEW] use _rocketchatLogger in Apps debugLog (#21000) Type check useTranslation hook Type check useEndpoint hook Fixed service lint errors Update server/services/team/service.ts fixed "describe.only" Changed async "before" functions to promise chain Fixed lint errors teams.delete api tests Basic test Fix service.ts formatting Fix code formatting Remove log Prevent last owner from leaving a team add tests for permission Added a param to receive a list of rooms to remove along with the team fix wrong roomId on groups.convertToTeam ...
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)
Change the
console.logcomment inorchestrator.jstothis.getRocketChatLogger().debugso that the logs can be disabled if the log level setting on the Admin panel is set to 0 or 1Issue(s)
Closes RocketChat/Rocket.Chat.Apps-engine#388
Steps to test or reproduce
Further comments