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

refactor: delete templateAjax #28563

Closed
jdotcms opened this issue May 14, 2024 · 1 comment · Fixed by #28572
Closed

refactor: delete templateAjax #28563

jdotcms opened this issue May 14, 2024 · 1 comment · Fixed by #28572
Assignees
@jdotcms
Labels
QA : Not Needed Team : Lunik Triage Type : Task

Comments

@jdotcms
Copy link
Contributor

jdotcms commented May 14, 2024

Parent Issue

No response

Task

TemplateAjax is not longer used, lets remove it in favor of TemplateResource

Proposed Objective

Core Features

Proposed Priority

Priority 3 - Average

Acceptance Criteria

The template ajax wont exit

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

No response

Assumptions & Initiation Needs

No response

Quality Assurance Notes & Workarounds

Do not need QA

Sub-Tasks & Estimates

No response
jdotcms added a commit that referenced this issue May 14, 2024
@jdotcms
#28563 removing template ajax
897266c
@jdotcms jdotcms linked a pull request May 14, 2024 that will close this issue
fix: #28563 removing template ajax #28572
Merged
jdotcms added a commit that referenced this issue May 15, 2024
@jdotcms
#28563 removing template ajax
9f5683c
jdotcms added a commit that referenced this issue May 15, 2024
@jdotcms
#28563 removing template ajax
d4727d7
jdotcms added a commit that referenced this issue May 15, 2024
@jdotcms
#28563 removing template ajax
8328392
github-merge-queue bot pushed a commit that referenced this issue May 15, 2024
@jdotcms
fix: #28563 removing template ajax (#28572)
ce2bfa8 
Just removing unused ajax
jdotcms added a commit that referenced this issue May 15, 2024
@jdotcms
Revert "fix: #28563 removing template ajax (#28572)"
cf827da 
This reverts commit ce2bfa8.
jdotcms added a commit that referenced this issue May 17, 2024
@jdotcms
#28563 nosonar
5f65661
github-merge-queue bot pushed a commit that referenced this issue May 18, 2024
@jdotcms @fmontes @erickgonzalez
Revert "fix: #28563 removing template ajax (#28572)" (#28582)
32f41ee 
This reverts commit ce2bfa8.

---------

Co-authored-by: Freddy Montes <[email protected]>
Co-authored-by: erickgonzalez <[email protected]>
@rjvelazco rjvelazco assigned rjvelazco and jdotcms and unassigned rjvelazco May 20, 2024
jdotcms added a commit that referenced this issue May 20, 2024
@jdotcms
#28563 some progress
b0bf1bc
jdotcms added a commit that referenced this issue May 20, 2024
@jdotcms
#28563 adding first draft to totally remove the TemplateAjax
325919e
@jdotcms
Copy link
Contributor Author

jdotcms commented May 21, 2024

change the effort to 5, since the issue gets more complicated since the javascript references where not considered on the initial estimation

jdotcms added a commit that referenced this issue May 22, 2024
@jdotcms
#28563 removing template ajax
90565d7
jdotcms added a commit that referenced this issue May 22, 2024
@jdotcms
#28563 final code for removing template ajax
ea93c57
jdotcms added a commit that referenced this issue May 22, 2024
@jdotcms
#28563 feedback
986a66b
jdotcms added a commit that referenced this issue May 28, 2024
@jdotcms
#28563 adding feedback
0a0bbb3
jdotcms added a commit that referenced this issue May 28, 2024
@jdotcms
#28563 adding feedback
bdf3bd7
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jdotcms jdotcms

Labels
QA : Not Needed Team : Lunik Triage Type : Task
Projects
dotCMS - Product Planning
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: #28563 removing template ajax dotCMS/core
2 participants
@jdotcms @rjvelazco