-
-
Notifications
You must be signed in to change notification settings - Fork 257
Refresh pages #674
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
Refresh pages #674
Conversation
Replaces SvelteKit setup with Astro, removing SvelteKit-specific files, configs, and routes. Adds Astro configuration, new Astro pages, and updates project structure to use Astro conventions. Moves static assets, updates .gitignore and Prettier config, and removes ESLint and SvelteKit dependencies.
Created a new Ubuntu installation page with a placeholder for instructions. Updated the index page to use 'Ubuntu' and 'Flatpak' for consistency. Minor formatting adjustment in the macOS page.
Created a new page with installation instructions for OBS Background Removal on Arch Linux. Improved formatting and clarity on the Flatpak installation page.
Added multiple favicon and manifest image files for improved browser and device support. Updated Layout.astro to include new favicon links using BASE_URL. Added a new endpoint at metadata/latest-version.txt.ts to serve the latest release version from GitHub.
Added a scheduled trigger to the GitHub Actions workflow to run daily at 2:43 AM. Simplified getLatestRelease by removing event-based logic and always fetching the latest release from the GitHub API.
Updated getLatestRelease to include an Authorization header if GITHUB_TOKEN is set, improving API rate limit handling. Also updated the build step in the workflow to use npx astro build with site and base path arguments.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR migrates the pages site from SvelteKit to Astro, redesigns the pages, and streamlines installation documentation by removing outdated platform-specific guides while adding a new metadata endpoint.
- Replaces SvelteKit framework with Astro for simpler static site generation
- Removes Windows, Ubuntu, Fedora, and openSUSE installation pages (to be re-added later)
- Adds
/metadata/latest-version.txtendpoint for improved update checking
Reviewed Changes
Copilot reviewed 50 out of 68 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| pages/package.json | Updates dependencies to use Astro instead of SvelteKit and removes ESLint/Prettier plugins |
| pages/astro.config.mjs | Adds Astro configuration file |
| pages/tsconfig.json | Simplifies TypeScript config to extend Astro's strict preset |
| pages/src/pages/*.astro | Creates new Astro pages replacing Svelte components for index, usage, Windows, macOS, Ubuntu, Flatpak, and Arch |
| pages/src/lib/github.ts | Adds new GitHub API client with TypeScript interfaces for release data |
| pages/src/pages/metadata/latest-version.txt.ts | Implements new endpoint returning latest release version as plain text |
| pages/src/layouts/Layout.astro | Creates base layout with HTML structure and global styles |
| pages/src/styles/white-box.css | Adds new styling for main content container |
| .github/workflows/pages.yaml | Updates build workflow for Astro, adds scheduled daily rebuild |
Co-authored-by: Copilot <[email protected]>
Corrects the project name from 'obs-backgroundremova-pages' to 'obs-backgroundremoval-pages' in package files and updates the GitHub API URL in github.ts to point to the correct repository for fetching the latest release.
…s-backgroundremoval into umireon/refresh-pages
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 50 out of 68 changed files in this pull request and generated 2 comments.
Co-authored-by: Copilot <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 50 out of 68 changed files in this pull request and generated 2 comments.
royshil
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a great change
* Migrate from SvelteKit to Astro and restructure project Replaces SvelteKit setup with Astro, removing SvelteKit-specific files, configs, and routes. Adds Astro configuration, new Astro pages, and updates project structure to use Astro conventions. Moves static assets, updates .gitignore and Prettier config, and removes ESLint and SvelteKit dependencies. * Create macos.astro * Add Ubuntu page and update OS names in index Created a new Ubuntu installation page with a placeholder for instructions. Updated the index page to use 'Ubuntu' and 'Flatpak' for consistency. Minor formatting adjustment in the macOS page. * Create flatpak.astro * Add Arch Linux install guide and update Flatpak page Created a new page with installation instructions for OBS Background Removal on Arch Linux. Improved formatting and clarity on the Flatpak installation page. * Update index.astro * Add favicons and latest version endpoint Added multiple favicon and manifest image files for improved browser and device support. Updated Layout.astro to include new favicon links using BASE_URL. Added a new endpoint at metadata/latest-version.txt.ts to serve the latest release version from GitHub. * Update workflow schedule and simplify release fetching Added a scheduled trigger to the GitHub Actions workflow to run daily at 2:43 AM. Simplified getLatestRelease by removing event-based logic and always fetching the latest release from the GitHub API. * Add GITHUB_TOKEN auth to GitHub API requests Updated getLatestRelease to include an Authorization header if GITHUB_TOKEN is set, improving API rate limit handling. Also updated the build step in the workflow to use npx astro build with site and base path arguments. * Update pages.yaml * Update pages/src/pages/usage.astro Co-authored-by: Copilot <[email protected]> * Fix project name and update GitHub release URL Corrects the project name from 'obs-backgroundremova-pages' to 'obs-backgroundremoval-pages' in package files and updates the GitHub API URL in github.ts to point to the correct repository for fetching the latest release. * Update macos.astro * Update windows.astro * Update ubuntu.astro * Apply suggestions from code review Co-authored-by: Copilot <[email protected]> --------- Co-authored-by: Copilot <[email protected]>
Closes #673