Skip to content

chore: migrate tfl-bus-status to standalone repo#874

Merged
514sid merged 2 commits into
masterfrom
chore/remove-tfl-bus-status
Jun 29, 2026
Merged

chore: migrate tfl-bus-status to standalone repo#874
514sid merged 2 commits into
masterfrom
chore/remove-tfl-bus-status

Conversation

@nicomiguelino

@nicomiguelino nicomiguelino commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

User description

Removes source files from edge-apps/tfl-bus-status and redirects to Screenly/tfl-bus-status-app. Closes #842.


PR Type

Documentation, Other


Description

  • Redirects app to standalone repository

  • Removes migrated source and manifests

  • Updates root README app link


Diagram Walkthrough

flowchart LR
  A["Playground TFL Bus Status"] -- "source removed" --> B["Standalone repository"]
  C["Playground README"] -- "links to" --> B
  D["App README"] -- "redirects users" --> B
Loading

File Walkthrough

Relevant files
Code removal
4 files
script.js
Removes TFL bus app JavaScript logic                                         
+0/-463 
common.css
Removes shared TFL app styling                                                     
+0/-306 
style.css
Removes responsive TFL app styles                                               
+0/-982 
index.html
Removes TFL app HTML entrypoint                                                   
+0/-72   
Documentation
2 files
README.md
Updates TFL app repository link                                                   
+1/-1     
README.md
Replaces guide with repository redirect                                   
+2/-61   
Configuration changes
2 files
screenly.yml
Removes production Screenly app manifest                                 
+0/-44   
screenly_qc.yml
Removes QC Screenly app manifest                                                 
+0/-44   

- remove source files from edge-apps/tfl-bus-status
- redirect README to Screenly/tfl-bus-status-app
- update root README entry
@github-actions

github-actions Bot commented Jun 26, 2026

Copy link
Copy Markdown

PR Reviewer Guide 🔍

(Review updated until commit 55336eb)

Here are some key observations to aid the review process:

🎫 Ticket compliance analysis ✅

842 - PR Code Verified

Compliant requirements:

  • edge-apps/tfl-bus-status source files and manifests are removed from Playground.
  • Users are directed to Screenly/tfl-bus-status-app from the app README.
  • The root README now links the TFL Bus Status App to the standalone repository.

Requires further human verification:

  • Confirm that the standalone Screenly/tfl-bus-status-app repository contains the migrated source, assets, manifests, and any required setup documentation.
⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ No major issues detected

@github-actions

Copy link
Copy Markdown

PR Code Suggestions ✨

No code suggestions found for the PR.

@nicomiguelino nicomiguelino marked this pull request as ready for review June 27, 2026 05:06
@github-actions

Copy link
Copy Markdown

Persistent review updated to latest commit 55336eb

@github-actions

Copy link
Copy Markdown

PR Code Suggestions ✨

No code suggestions found for the PR.

514sid
514sid previously approved these changes Jun 29, 2026
@salmanfarisvp salmanfarisvp self-requested a review June 29, 2026 09:17
@514sid 514sid merged commit dd5affe into master Jun 29, 2026
4 checks passed
@514sid 514sid deleted the chore/remove-tfl-bus-status branch June 29, 2026 09:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

chore: migrate edge-apps/tfl-bus-status to standalone repo (Screenly/tfl-bus-status-app)

3 participants