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

Rewrite ThreadUtils #24714

Closed
wezell opened this issue Apr 21, 2023 · 1 comment · Fixed by #24715
Closed

Rewrite ThreadUtils #24714

wezell opened this issue Apr 21, 2023 · 1 comment · Fixed by #24715
Labels
Merged QA : Not Needed Release : 23.06 stale Team : Scout Type : New Functionality

Comments

@wezell
Copy link
Contributor

wezell commented Apr 21, 2023

Parent Issue

Tech Debt

User Story

The current ThreadUtils was cut and pasted from another project and has many non-useful methods. We need to rewrite it to make it cleaner.

Acceptance Criteria

The ThreadUtils class should be rewritten to only implement the methods we use in dotCMS

Proposed Objective

Security & Privacy

Proposed Priority

Priority 2 - Important

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 Apr 21, 2023
@wezell
#24714 clean rewrite of ThreadUtils
e95d651
@wezell wezell linked a pull request Apr 24, 2023 that will close this issue
#24714 clean rewrite of ThreadUtils #24715
Merged
@fabrizzio-dotCMS fabrizzio-dotCMS self-assigned this Apr 25, 2023
fabrizzio-dotCMS added a commit that referenced this issue Apr 26, 2023
@fabrizzio-dotCMS
#24714 small test
abfc8a9
fabrizzio-dotCMS added a commit that referenced this issue Apr 26, 2023
@fabrizzio-dotCMS
#24714 merge with master
4217d8c
fabrizzio-dotCMS added a commit that referenced this issue Apr 26, 2023
@fabrizzio-dotCMS
#24714
3e81c6f
fabrizzio-dotCMS added a commit that referenced this issue Apr 26, 2023
@fabrizzio-dotCMS
#24714
9c418d5
nollymar pushed a commit that referenced this issue Apr 26, 2023
@wezell @fabrizzio-dotCMS
#24714 clean rewrite of ThreadUtils (#24715)
38985f6 
* #24714 clean rewrite of ThreadUtils

* #24714 small test

* #24714

* #24714

---------

Co-authored-by: fabrizzio-dotCMS <fabrizzio@dotCMS.com>
@nollymar nollymar reopened this Apr 26, 2023
@github-actions
Copy link

This issue is stale because it has been open 90 days with no activity. Remove stale label or comment or this will be closed in 30 days.

@github-actions github-actions bot added the stale label Jul 31, 2023
@wezell wezell closed this as completed Nov 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Merged QA : Not Needed Release : 23.06 stale Team : Scout Type : New Functionality
Projects
dotCMS - Product Planning
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

#24714 clean rewrite of ThreadUtils dotCMS/core
4 participants
@wezell @dsilvam @nollymar @fabrizzio-dotCMS