Skip to content

Conversation

@joaquinVill
Copy link
Contributor

@joaquinVill joaquinVill commented Jan 7, 2026

Everything went pretty smoothly with this update. The only issue I came across was that Angular 18’s stricter TS checks now flag using injected services in field initializers. I had to move UntypedFormGroup construction into the constructor so the dependent services are initialized before the forms are created.

@joaquinVill joaquinVill changed the base branch from main to chore/sc-36141/upgrade-angular-from-16-to-17-on-clark-client January 15, 2026 20:30
@joaquinVill joaquinVill marked this pull request as ready for review January 15, 2026 20:32
@joaquinVill joaquinVill requested review from a team, Cwagne17, curlyLasagna and dsoto18 January 15, 2026 20:32
dsoto18 and others added 22 commits January 22, 2026 13:55
…eci-config-yml-for-clark-client

upload source maps moved under jobs
…t-rating-error-message

rating character limit
…-deployment-1-22-26

feature/sc-38598/clark-client-deployment-1-22-26
This was painful. Client really needs a rewrite
Add an actions column for the preview and download functionality
…t-when-the-learning-object

chore/sc-38582/disable-chatbot-when-the-learning-object
…le-previewer-for-microsoft

Implement file previewer for Office files
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants