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

installer: nginx: add missing locales location required by ui_next #1822

Merged
merged 1 commit into from
May 1, 2024

Conversation

sinuscosinustan
Copy link
Contributor

@sinuscosinustan sinuscosinustan commented Apr 12, 2024

SUMMARY

The new UI depends on the locales available via ingress/locales which is being served as static file and not via Django.

A change in the nginx.conf was already done for the dev environment in commit ec4f10d86881389af12371f90cb75af03417d109 (AWX), but not here.

ISSUE TYPE
  • Bug, Docs Fix or other nominal change

@sinuscosinustan sinuscosinustan changed the title installer: nginx: add missing locales location for ui_next installer: nginx: add missing locales location required by ui_next Apr 12, 2024
@sinuscosinustan sinuscosinustan force-pushed the ts-fix-locales branch 3 times, most recently from 703b007 to 83f3d86 Compare April 18, 2024 09:27
The new UI depends on the locales available via ingress/locales which
is being served as static file and not via Django.

A change in the nginx.conf was already done for the dev environment
in commit ec4f10d86881389af12371f90cb75af03417d109 (AWX), but not
here.

Signed-off-by: Tom Siewert <[email protected]>
@sinuscosinustan
Copy link
Contributor Author

@rooftopcellist PR Check failed with The PR body must contain exactly one of [ 'Bug, Docs Fix or other nominal change', 'New or Enhanced Feature', 'Breaking Change' ], but it contains a Bug fix in the body. Any idea why it fails?

@rooftopcellist
Copy link
Member

rooftopcellist commented May 1, 2024

@tomsiewert Thanks for the PR! Yes, CI was complaining because it expects 3 categories, the entire category is Bug, Docs Fix or other nominal change, not just Bug Fix. This is part of our automation for generating release notes. Admittedly, this can be a bit cumbersome. Sorry for the confusion!

I edited the description, it should pass now, I'll merge shortly.

@rooftopcellist rooftopcellist merged commit 6fff7cb into ansible:devel May 1, 2024
7 of 8 checks passed
@sinuscosinustan sinuscosinustan deleted the ts-fix-locales branch May 2, 2024 11:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants