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

Merge involved authorities preparation #339

Merged
merged 5 commits into from
May 10, 2024

Conversation

damisul
Copy link
Collaborator

@damisul damisul commented May 7, 2024

This small PR contains two commits from merge_involved_authorities branch.
I believe it would be better to merge them separately from other code, to reduce amount of code conflicts to be resolved:

  • Simple Form gem addition
  • Some deprecated code cleanup

Also it contains some adjustemets to configuration:

  • removed deprecated i18n setting in test env.
  • removed several empty specs generated by scaffold.
  • added setting to elasticsearch docker config to avoid seeing warnings during specs

@abartov abartov merged commit 960f5c4 into master May 10, 2024
7 checks passed
@abartov abartov deleted the merge_involved_authorities_preparation branch May 10, 2024 15:27
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.

2 participants