Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: slow response in email composer #510

Open
wants to merge 2 commits into
base: devel
Choose a base branch
from

Conversation

mananpatel13589
Copy link

@mananpatel13589 mananpatel13589 commented Sep 23, 2024

Ref : CO-1454

@CLAassistant
Copy link

CLAassistant commented Sep 23, 2024

CLA assistant check
All committers have signed the CLA.

This comment has been minimized.

Copy link

Passed

Analysis Details

0 Issues

  • Bug 0 Bugs
  • Vulnerability 0 Vulnerabilities
  • Code Smell 0 Code Smells

Coverage and Duplications

  • Coverage 0.00% Coverage (65.80% Estimated after merge)
  • Duplications 0.00% Duplicated Code (0.00% Estimated after merge)

Project ID: carbonio-shell-ui

View in SonarQube

@mananpatel13589 mananpatel13589 marked this pull request as ready for review September 24, 2024 06:38
Copy link
Contributor

@beawar beawar left a comment

Choose a reason for hiding this comment

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

We tried your solution with the email attached in the issue.
Unfortunately, we cannot see the performance improvement with these changes.
By seeing the documentation and the code, we think that a possible improvement could be reached by using the uncontrolled mode.

cc @CMazzilli

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