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

Refactor/replace dynamic locales #37

Merged
merged 6 commits into from
Jul 11, 2023

Conversation

queuing4oranges
Copy link
Collaborator

  • updated asab-webui in seacat-auth-webui
  • no changes in seacat-auth-webui/public/locales as there is no dynamic locales
  • correction of double classname

@queuing4oranges queuing4oranges self-assigned this Jul 6, 2023
CHANGELOG.md Outdated
@@ -4,7 +4,7 @@

### Features

- ASAB WebUI submodule version bump [[ae64c84](https://github.com/TeskaLabs/asab-webui/commit/ae64c84a78ecbbc32a1a02c5bf3bc196907b0263)] commit (INDIGO Sprint 230526, [!32](https://github.com/TeskaLabs/seacat-auth-webui/pull/32))
- ASAB WebUI submodule version bump [[297aa81](https://github.com/TeskaLabs/asab-webui/commit/297aa819e97964aec33e03f545fc10819f0fe325)] commit (INDIGO Sprint 230623, [!37](https://github.com/TeskaLabs/seacat-auth-webui/pull/37))
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hmmms.. is this really the latest master version of asab-webui submodule?... 🤔

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@queuing4oranges Petr is right, please update the CHANGELOG and also asab webui submodule to this commit:
TeskaLabs/asab-webui@06af558

CHANGELOG.md Outdated
@@ -4,7 +4,7 @@

### Features

- ASAB WebUI submodule version bump [[ae64c84](https://github.com/TeskaLabs/asab-webui/commit/ae64c84a78ecbbc32a1a02c5bf3bc196907b0263)] commit (INDIGO Sprint 230526, [!32](https://github.com/TeskaLabs/seacat-auth-webui/pull/32))
- ASAB WebUI submodule version bump [[297aa81](https://github.com/TeskaLabs/asab-webui/commit/297aa819e97964aec33e03f545fc10819f0fe325)] commit (INDIGO Sprint 230623, [!37](https://github.com/TeskaLabs/seacat-auth-webui/pull/37))
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@queuing4oranges Petr is right, please update the CHANGELOG and also asab webui submodule to this commit:
TeskaLabs/asab-webui@06af558

@petrKavulok petrKavulok marked this pull request as ready for review July 7, 2023 09:38
Copy link
Contributor

@petrKavulok petrKavulok left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Please, resolve conflicts and it's good to go :)

Copy link
Contributor

@petrKavulok petrKavulok left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. 👍 🔥

@queuing4oranges queuing4oranges requested a review from Pe5h4 July 7, 2023 10:46
Copy link
Collaborator

@Pe5h4 Pe5h4 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gut!

@queuing4oranges queuing4oranges merged commit bb507c6 into main Jul 11, 2023
@queuing4oranges queuing4oranges deleted the refactor/replace-dynamic-locales branch July 11, 2023 08:30
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.

3 participants