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

dotcms/editor-content: Use the new Angular Syntax #29625

Closed
Tracked by #29636
, #29048
...
nicobytes opened this issue Aug 16, 2024 · 2 comments · Fixed by #29626
Closed
Tracked by #29636
, #29048
...

dotcms/editor-content: Use the new Angular Syntax #29625

nicobytes opened this issue Aug 16, 2024 · 2 comments · Fixed by #29626
Labels
Gardening Day Merged QA : Approved Release : 24.08.27 Custom templates and loggin fixes Team : Falcon Type : Task

Comments

@nicobytes
Copy link
Contributor

nicobytes commented Aug 16, 2024 •
edited
Loading

Parent Issue

#29048

Task

Using the command nx g @angular/core:control-flow to migrate to the new Angular syntax

Proposed Objective

Code Maintenance

Proposed Priority

Priority 4 - Trivial

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
@nicobytes nicobytes self-assigned this Aug 16, 2024
@nicobytes nicobytes moved this from New to In Progress in dotCMS - Product Planning Aug 16, 2024
@nicobytes nicobytes linked a pull request Aug 16, 2024 that will close this issue
chore(edit-content): use the new angular syntax #29626
Merged
3 tasks
@nicobytes nicobytes mentioned this issue Aug 16, 2024
Merged
3 tasks
@nicobytes
Copy link
Contributor Author

Manual test to edit content

2024-08-16.17-09-48.mp4

@nicobytes nicobytes moved this from In Progress to In Review in dotCMS - Product Planning Aug 16, 2024
This was referenced Aug 19, 2024
Closed
Open
@nicobytes nicobytes changed the title Editor Content: Use the new Angular Syntax dotcms/editor/content: Use the new Angular Syntax Aug 19, 2024
@nicobytes nicobytes changed the title dotcms/editor/content: Use the new Angular Syntax dotcms/editor-content: Use the new Angular Syntax Aug 19, 2024
github-merge-queue bot pushed a commit that referenced this issue Aug 19, 2024
@nicobytes
chore(edit-content): use the new angular syntax (#29626)
7212cb8 
### Parent Issue
#29625

### Task
Using the command `nx g @angular/core:control-flow `to migrate to the
new Angular syntax

### Proposed Objective
Use the new Angular syntax to improve performance and code
maintainability.

### Checklist
- [x] Tests
- [x] Translations
- [x] Security Implications Contemplated (add notes if applicable)
@github-project-automation github-project-automation bot moved this from In Review to Internal QA in dotCMS - Product Planning Aug 19, 2024
@nicobytes nicobytes removed their assignment Aug 20, 2024
@josemejias11
Copy link
Contributor

Approved: Tested on ** trunk_fa74e31**, Docker, macOS 14.5, FF v126.0.1

@josemejias11 josemejias11 added QA : Approved Release : 24.08.27 Custom templates and loggin fixes labels Aug 23, 2024
@josemejias11 josemejias11 moved this from QA - In Progress to Done in dotCMS - Product Planning Aug 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Gardening Day Merged QA : Approved Release : 24.08.27 Custom templates and loggin fixes Team : Falcon Type : Task
Projects
dotCMS - Product Planning
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

chore(edit-content): use the new angular syntax dotCMS/core
2 participants
@nicobytes @josemejias11