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.

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

Remove AngularCookieLocaleResolver #1444

Merged

Conversation

shrralis
Copy link
Contributor

remove AngularCookieLocaleResolver due to being a leftover from angularjs, a companion of jhipster/generator-jhipster#23818

Fix jhipster/generator-jhipster#20106

@shrralis shrralis marked this pull request as ready for review October 11, 2023 20:44
@shrralis
Copy link
Contributor Author

I suppose those checks should start passing after jhipster/generator-jhipster#23818 gets merged?

@DanielFran
Copy link
Member

@shrralis Yes
The pipeline target main brach and not your branch, this is why it is failing here...
But do not worry, this will be merged only if on the generator side everything is working and validated

@shrralis shrralis closed this Apr 30, 2024
@shrralis shrralis force-pushed the fix-20106-drop-AngularCookieLocaleResolver branch from 22786f4 to a069f04 Compare April 30, 2024 06:57
remove AngularCookieLocaleResolver due to be a leftover from angularjs

Fix #20106
@shrralis shrralis reopened this Apr 30, 2024
@DanielFran DanielFran merged commit ad80ae6 into jhipster:main Apr 30, 2024
40 of 49 checks passed
@DanielFran
Copy link
Member

Thanks @shrralis for the contribution

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.

Drop AngularCookieLocaleResolver or rename
2 participants