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

[conflict_resolver] Resolved tab load speed #9488

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

victori444
Copy link
Contributor

@victori444 victori444 commented Nov 27, 2024

Brief summary of changes

Fixes load speed of the CR's resolved conflicts tab that was failing because of too much data.

  • Queries for the projectid and centerid while keeping their full names in the filter for Project & Site
  • Replaces fields "Value 1" and "Value 2" with "Incorrect Value"

Testing instructions (if applicable)

  1. Check that both resolved/unresolved tabs of the module load properly
  2. In the resolved tab, check that the Project & Site filters works
  3. Verify that "Value 1" and "Value 2" fields in the filter and table have been replaced with "Incorrect Value", and that the filter works well

CCNA Override

@maximemulder maximemulder added Project: CCNA Issue or PR related to the CCNA project Category: Perfomance Issue or PR that aims to report or improve perfomance and removed Priority: Projects labels Nov 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Category: Perfomance Issue or PR that aims to report or improve perfomance Project: CCNA Issue or PR related to the CCNA project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants