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

Blog API: Fix locale, routes and psalm errors #598

Merged
merged 2 commits into from
Oct 6, 2023
Merged

Blog API: Fix locale, routes and psalm errors #598

merged 2 commits into from
Oct 6, 2023

Conversation

vjik
Copy link
Member

@vjik vjik commented Oct 6, 2023

Q A
Is bugfix? ✔️
New feature?
Breaks BC?
Fixed issues -

@what-the-diff
Copy link

what-the-diff bot commented Oct 6, 2023

PR Summary

  • Enhanced Code Formatting Rules
    Created a configuration file .editorconfig to standardize and improve formatting rules for PHP files located in the blog-api folder. Also, updated the system to retain whitespaces at the end of .md files.

  • Simplifying Container Management and Test Execution
    Introduced a Makefile in the blog-api folder that provides commands to simplify interactions with Docker, execute composer commands, run Yii commands, and conduct tests.

  • Improved URL Management
    Updated router.php in the blog-api/config/common/di directory. Introduced a new class called UrlGenerator to streamline URL generation and made enhancements to the class properties and methods.

  • Locale Setting Event Handling
    Added events.php in the blog-api/config/web directory to handle the locale setting event (SetLocaleEvent), improving localization features.

  • Expanded Configuration Options
    Modified the configuration.php file in the blog-api directory. Added a events-web key for more extensive configuration possibilities.

  • Removed Redundant Bootstrap Installation
    Updated the Makefile in the blog directory to remove the redundant command of npm for installing Bootstrap 5.3.1.

@vjik vjik changed the title Blog API: Fix locale and routes Blog API: Fix locale, routes and psalm errors Oct 6, 2023
@vjik vjik merged commit 561265e into master Oct 6, 2023
15 checks passed
@vjik vjik deleted the fix-blog-api branch October 6, 2023 11:46
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.

1 participant