Skip to content

Add home redirection#3437

Merged
ildyria merged 1 commit intomasterfrom
home-redirect
Jun 14, 2025
Merged

Add home redirection#3437
ildyria merged 1 commit intomasterfrom
home-redirect

Conversation

@ildyria
Copy link
Copy Markdown
Member

@ildyria ildyria commented Jun 12, 2025

This PR is a precondition for the future timeline page, feed page (and others?)

This pull request introduces a new "Home" page feature, allowing users to configure a default homepage for the application. The changes span backend configuration, frontend routing, and state management updates. Below is a summary of the most important changes grouped by theme:

Backend Changes:

  • Added a new database migration (2025_06_12_102340_home_page.php) to define a configuration key home_page_default with a default value of "gallery" for setting the default homepage.
  • Updated InitConfig class to include a new property default_homepage and fetch its value from the configuration. [1] [2]

Frontend Routing and Components:

  • Introduced a new Home.vue component that redirects users to the appropriate default homepage based on the configuration.
  • Updated the frontend router (routes.ts) to add a route for the new "Home" page (/home). [1] [2]
  • Modified Landing.vue and Statistics.vue to redirect to the "Home" page instead of the "Gallery" page. [1] [2] [3] [4]

State Management:

  • Updated LycheeStateStore to include a new default_homepage property and ensure it is initialized when loading application data. [1] [2]

Miscellaneous:

  • Adjusted the web_v2.php routes file to add a /home route for the new homepage.
  • Removed the "home" keyword from the honeypot configuration to avoid conflicts with the new route.

@ildyria ildyria requested a review from a team as a code owner June 12, 2025 08:41
@ildyria ildyria added the Review: easy Easy review expected: probably just need a quick to go through. label Jun 12, 2025
@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 12, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 86.66%. Comparing base (b71c206) to head (5a43151).
Report is 2 commits behind head on master.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ildyria ildyria added the ignore-for-release Do not publish this PR in the release summary. label Jun 14, 2025
@ildyria ildyria merged commit 74de275 into master Jun 14, 2025
35 checks passed
@ildyria ildyria deleted the home-redirect branch June 14, 2025 10:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ignore-for-release Do not publish this PR in the release summary. Review: easy Easy review expected: probably just need a quick to go through.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants