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

Dev #307

Closed
wants to merge 2 commits into from
Closed

Dev #307

wants to merge 2 commits into from

Conversation

goghh
Copy link

@goghh goghh commented Sep 14, 2023

Q A
Is bugfix? yes/no
New feature? yes/no
Breaks BC? yes/no
Tests pass? yes/no
Fixed issues comma-separated list of tickets # fixed by the PR, if any

@what-the-diff
Copy link

what-the-diff bot commented Sep 14, 2023

PR Summary

  • Addition of Docker and environment configuration files
    New files for Docker, xdebug, and the environment variables have been included to help fine-tune the development environment setup. This aids in maintaining a uniform development environment across all machines.

  • File structure and dependency modifications
    Some outdated or unnecessary files have been removed while a few others have been updated. This is to emphasize more modern coding practices and remove old, out-of-date components.

  • Database Management Improvements
    The database configuration files have been updated to receive credentials from environment variables. This helps in maintaining the security and flexibility of the database configurations.

  • News Management System Implementation
    New classes for handling news-related functionalities have been added. These include creating, deleting, and retrieving news data, and other related activities. It also includes changes to handle potentially inappropriate content and manage news entries more effectively.

  • Frontend Configuration with Docker
    The frontend folder has been restructured with new Docker files and environment variables. It makes for easier deployment and a more efficient development process.

  • React Project Setup
    The project now uses React for its frontend, which includes a new file hierarchy for a React-based application.

  • Removal of Unused Components
    Old components, views, and test files that aren't relevant to the new structure of the application have been removed. This clears up the codebase and avoids any possible confusion due to leftover legacy code.

  • News Content and Handling Overhaul
    The news handling service has been entirely restructured, with new functionality for removing inappropriate content and maintaining a cleaner news feed. This helps ensure that the news feed remains user-friendly and relevant.

  • Introduction of New Models
    New models for news and obscene words have been added, replacing the older ContactForm and LoginForm models.

  • NewsService improvements
    Within the services/NewsService, we have introduced several new components responsible for managing, moderating, and performing CRUD operations on news items. It keeps the architecture clean and separate for easier modification, testing, and understanding.

  • File Deletions
    Several files from the views, test, controllers, and widgets folder have been removed. These were unnecessary or unused in the current version of the application. Careful deletion improves the overall project organisation and clarity.

  • Test Cleanup
    A few older test cases that no longer applied to this version of the application have been removed. This ensures that our test suites remain relevant and accurate.

@goghh goghh closed this Sep 15, 2023
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