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

Fix after clean install and misc #392

Draft
wants to merge 9 commits into
base: develop
Choose a base branch
from

Conversation

neznaika0
Copy link
Contributor

@neznaika0 neznaika0 commented Sep 4, 2024

Sorry, for one PR. It's easier to apply the changes this way. The excess can be returned to its original state.

these are general fixes that are found after a clean installation and exemplary user behavior.

  • fix: Add id for HTMX action
    "Edit Thread" button does not work
  • fix: Add LEFT JOIN instead of JOIN
    If there are no Categories or Threads/Posts inside, the list of Categories is empty. It should be displayed regardless of Trends/Posts
  • fix: Add URL for home page
    Clicking on the Logo/Sitename does not lead anywhere. Annoy
  • fix: Сase when no categories
    If there are no categories in the database, exceptions will occur. You need to add empty arrays by default
  • fix: Configure cache handler
    The initial setup does not provide for the use of a Cache. Corrected .env
  • fix: Delete unnecessary namespaces
  • fix: Bad encoding DOMDocument
    The texts were displayed incorrectly after formatting. See https://www.php.net/manual/en/domdocument.loadhtml.php#74777
    EDIT: HTML-ENTITIES will be deprecated. See https://www.php.net/manual/en/function.mb-convert-encoding.php#127529
  • Hide pagination if total pages = 1

@neznaika0 neznaika0 force-pushed the fix-after-clean-install branch from ec7d74c to 60ea4f4 Compare September 4, 2024 14:28
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