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.

Sign up for GitHub

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

Jump to bottom

remove unneeded DB hit from edit content screen #27187

Closed
wezell opened this issue Jan 5, 2024 · 1 comment
Closed

remove unneeded DB hit from edit content screen #27187

wezell opened this issue Jan 5, 2024 · 1 comment
Labels
Release : 23.01.14 Included in LTS patch release 23.01.14 Release : 23.10.24 v7 Included in LTS patch release 23.10.24 v7 Team : Bug Fixers Triage Type : Task

Comments

@wezell
Copy link
Contributor

wezell commented Jan 5, 2024

Parent Issue

No response

Task

On the edit content screen we call for a list of 100 structures that are never used. This results in an unneeded db hit every time and can be removed.

List<Structure> structures = StructureFactory.getStructuresByUser(user, "", "name", 100, 0,"asc");

Proposed Objective

Application Performance

Proposed Priority

Priority 3 - Average

Acceptance Criteria

No response

External Links... Slack Conversations, Support Tickets, Figma Designs, etc.

No response

Assumptions & Initiation Needs

No response

Quality Assurance Notes & Workarounds

No response

Sub-Tasks & Estimates

No response
wezell added a commit that referenced this issue Jan 5, 2024
@wezell
fix(perf) tiny change but can have big impact with regards to snappin…
c201b8c 
…ess on the edit_content screen ref:#27187
github-merge-queue bot pushed a commit that referenced this issue Jan 9, 2024
@wezell
fix(perf) tiny change but can have big impact with regards to snappin…
cf9b773 
…ess on the edit_content screen ref:#27187 (#27188)
@erickgonzalez
Copy link
Contributor

PR: #27188

@erickgonzalez erickgonzalez added the LTS : Next Ticket that will be added to LTS label Feb 28, 2024
@erickgonzalez erickgonzalez added Next LTS Release Shortlisted of issues that will be included in the upcoming LTS and removed LTS : Next Ticket that will be added to LTS labels Mar 15, 2024
erickgonzalez added a commit that referenced this issue Mar 15, 2024
@erickgonzalez
#27187 include in 23.01.14
29fab03
erickgonzalez added a commit that referenced this issue Mar 25, 2024
@erickgonzalez
#27187 include in 23.10.24
21ba4f1
@erickgonzalez erickgonzalez added Release : 23.10.24 v7 Included in LTS patch release 23.10.24 v7 Release : 23.01.14 Included in LTS patch release 23.01.14 and removed Next LTS Release Shortlisted of issues that will be included in the upcoming LTS labels Mar 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Release : 23.01.14 Included in LTS patch release 23.01.14 Release : 23.10.24 v7 Included in LTS patch release 23.10.24 v7 Team : Bug Fixers Triage Type : Task
Projects
dotCMS - Product Planning
Archived in project
Milestone
No milestone
Development

No branches or pull requests
2 participants
@wezell @erickgonzalez