Skip to content

docs: point Screenly migration guide at the in-app wizard - #3120

Merged
vpetersson merged 3 commits into
masterfrom
docs/migrate-to-screenly-in-app
Jul 6, 2026
Merged

docs: point Screenly migration guide at the in-app wizard#3120
vpetersson merged 3 commits into
masterfrom
docs/migrate-to-screenly-in-app

Conversation

@vpetersson

Copy link
Copy Markdown
Contributor

Issues Fixed

No associated issue. The docs described a removed migration path.

Description

The Anthias → Screenly asset migration is now a built-in wizard under Settings → Migrate to Screenly. The old tools/migrate_assets_to_screenly.py script it referenced no longer exists.

  • Rewrite the docs page for the in-app wizard flow (get a Screenly API token → Start migration → paste token → pick assets → migrate, with retry-on-failure)
  • Drop the removed SSH + migrate_assets_to_screenly.py script instructions, the deprecated HTTP-Basic-auth note, and the "Raspberry Pi OS only" caveat (the wizard is a standard Settings section, not device-gated)
  • Remove the stale migration link from the README Quick Links
  • Slug/alias unchanged, so the page URL and the repo redirect stub still resolve

Checklist

  • I have performed a self-review of my own code.
  • New and existing unit tests pass locally and on CI with my changes. (N/A — docs only)
  • I have done an end-to-end test for Raspberry Pi devices. (N/A — docs only)
  • I have tested my changes for x86 devices. (N/A — docs only)
  • I added a documentation for the changes I have made (when necessary).

- Rewrite the docs page for the built-in Settings → Migrate to Screenly flow
- Drop the removed SSH + migrate_assets_to_screenly.py script instructions
- Remove the stale migration link from the README Quick Links

Co-Authored-By: Claude Opus 4.8 (1M context) <[email protected]>
@vpetersson
vpetersson requested a review from a team as a code owner July 6, 2026 20:51
@vpetersson vpetersson self-assigned this Jul 6, 2026
@vpetersson
vpetersson requested a review from Copilot July 6, 2026 20:51

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the Anthias documentation to reflect that migrating assets to Screenly is now handled via an in-app Settings wizard (instead of the removed SSH/script-based flow), and removes an outdated quick-link from the README.

Changes:

  • Rewrites the “Migrating Assets to Screenly” docs page to describe the in-app wizard flow (token → select assets → migrate → retry failures).
  • Removes the old SSH/tools/migrate_assets_to_screenly.py script instructions and related deprecated notes/caveats.
  • Removes the stale migration link from the README “Quick Links”.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
website/content/docs/migrating-assets-to-screenly.md Replaces the removed script-based migration instructions with the current in-app wizard workflow.
README.md Removes an outdated quick-link to the migration docs.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread website/content/docs/migrating-assets-to-screenly.md Outdated
Comment thread website/content/docs/migrating-assets-to-screenly.md Outdated
- Reference the actual "Migrate N assets" and "Retry N failed" button text
- Keep the Retry label on a single line

Co-Authored-By: Claude Opus 4.8 (1M context) <[email protected]>

Copilot AI left a comment

Copy link
Copy Markdown

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 2 out of 2 changed files in this pull request and generated 2 comments.

Comment thread website/content/docs/migrating-assets-to-screenly.md Outdated
Comment thread website/content/docs/migrating-assets-to-screenly.md
- Clarify the API token is not stored on the device
- Add the "Get started → I have a token" step before the token entry

Co-Authored-By: Claude Opus 4.8 (1M context) <[email protected]>
@sonarqubecloud

sonarqubecloud Bot commented Jul 6, 2026

Copy link
Copy Markdown

Copilot AI left a comment

Copy link
Copy Markdown

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 2 out of 2 changed files in this pull request and generated no new comments.

@vpetersson
vpetersson merged commit 4c61d40 into master Jul 6, 2026
5 checks passed
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