Skip to content

Conversation

@d-gubert
Copy link
Member

@d-gubert d-gubert commented Jul 20, 2021

Proposed changes (including videos or screenshots)

  • Add appId field to the data saved by the Scheduler
  • Add appId index to rocketchat_apps_persistence model
  • Skip "trash collection" when deleting records from rocketchat_apps_persistence
  • Add a new setting to control for how long we should keep logs from the apps

image

image

Issue(s)

Steps to test or reproduce

Further comments

I'm not so certain about the setting name and the alert texts, inputs are very welcome :)

@d-gubert d-gubert requested a review from thassiov July 20, 2021 01:05
@d-gubert d-gubert added this to the 3.17.0 milestone Jul 20, 2021
Copy link
Contributor

@thassiov thassiov left a comment

Choose a reason for hiding this comment

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

LGTM!

@thassiov thassiov merged commit e135a29 into develop Jul 20, 2021
@thassiov thassiov deleted the apps/index-apps-persistence-model branch July 20, 2021 15:49
gabriellsh added a commit that referenced this pull request Jul 20, 2021
…improve/discussion-listimprove/discussion-listimprove/discussion-listimprove/discussion-listimprove/discussion-listimprove/discussion-listimprove/discussion-listimprove/discussion-listimprove/discussion-listimprove/discussion-listimprove/discussion-listimprove/discussion-listimprove/discussion-listimprove/discussion-listimprove/discussion-listimprove/discussion-listimprove/discussion-listimprove/discussion-listMerge branch 'develop' of github.com:RocketChat/Rocket.Chat into fix/attachment_emoji

* 'develop' of github.com:RocketChat/Rocket.Chat: (119 commits)
  [IMPROVE] Configuration for indices in Apps-Engine models (#22705)
  [FIX] Use room's last message time when visitor did not send any message (#22695)
  F room doesnt exists error when uploading file as first message (#22590)
  [IMPROVE] Refactor `livechat.registerGuest` function (#22684)
  change livechat_enabled default value to true (#22697)
  Chore: added pagination to search msg endpoint (#22632)
  Fixed crush in the getChannelHistory method (#22667)
  Regression: Fix ee microservices build (#22656)
  [IMPROVE] Rewrite sidebar footer as React Component (#22687)
  [FIX] Livechat config endpoint is not returning all settings (#22686)
  [FIX] Info page - Instances card modal table width #22627
  Bump version to 3.16.3
  fixed - room still listed after delete (#22613)
  Chore: Use projection instead of fields to avoid error log (#22629)
  [FIX] VisitorClientInfo not showing (#22593)
  Create README.md (#22615)
  [NEW] REST endpoint to remove User from Role (#20485)
  Bump version to 3.16.2
  Regression: Change the name of called methods in Users model (#22620)
  [FIX] Markdown for UiKit blocks (#22619)
  ...
gabriellsh added a commit that referenced this pull request Jul 20, 2021
…status-message

* 'develop' of github.com:RocketChat/Rocket.Chat: (114 commits)
  [FIX] Remove team's related actions inside of discussion #22470
  [IMPROVE] Configuration for indices in Apps-Engine models (#22705)
  [FIX] Use room's last message time when visitor did not send any message (#22695)
  F room doesnt exists error when uploading file as first message (#22590)
  [IMPROVE] Refactor `livechat.registerGuest` function (#22684)
  change livechat_enabled default value to true (#22697)
  Chore: added pagination to search msg endpoint (#22632)
  Fixed crush in the getChannelHistory method (#22667)
  Regression: Fix ee microservices build (#22656)
  [IMPROVE] Rewrite sidebar footer as React Component (#22687)
  [FIX] Livechat config endpoint is not returning all settings (#22686)
  [FIX] Info page - Instances card modal table width #22627
  Bump version to 3.16.3
  fixed - room still listed after delete (#22613)
  Chore: Use projection instead of fields to avoid error log (#22629)
  [FIX] VisitorClientInfo not showing (#22593)
  Create README.md (#22615)
  [NEW] REST endpoint to remove User from Role (#20485)
  Bump version to 3.16.2
  Regression: Change the name of called methods in Users model (#22620)
  ...
@ggazzo ggazzo mentioned this pull request Jul 31, 2021
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.

3 participants