Skip to content

chore: migrate charlie-hr-app to standalone repo#868

Merged
514sid merged 1 commit into
masterfrom
chore/remove-charlie-hr-app
Jun 29, 2026
Merged

chore: migrate charlie-hr-app to standalone repo#868
514sid merged 1 commit into
masterfrom
chore/remove-charlie-hr-app

Conversation

@nicomiguelino

@nicomiguelino nicomiguelino commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

User description

Removes source files from edge-apps/charlie-hr-app and redirects to Screenly/charlie-hr-app. Closes #822.


PR Type

Other, Documentation


Description

  • Migrates CharlieHR app out of monorepo

  • Replaces app README with repository redirect

  • Updates root app listing link

  • Removes legacy source and manifests


Diagram Walkthrough

flowchart LR
  playground["Playground monorepo"]
  stub["CharlieHR README redirect"]
  standalone["Screenly/charlie-hr-app"]
  playground -- "removes app source" --> stub
  stub -- "points to" --> standalone
Loading

File Walkthrough

Relevant files
Miscellaneous
3 files
main.js
Removed CharlieHR dashboard application logic                       
+0/-491 
styles.css
Removed CharlieHR dashboard styling                                           
+0/-323 
index.html
Removed CharlieHR dashboard HTML shell                                     
+0/-128 
Documentation
2 files
README.md
Redirected CharlieHR listing to standalone repository       
+1/-1     
README.md
Replaced app documentation with migration notice                 
+2/-71   
Configuration changes
2 files
screenly.yml
Removed production Screenly app manifest                                 
+0/-64   
screenly_qc.yml
Removed QC Screenly app manifest                                                 
+0/-64   

- remove source files from edge-apps/charlie-hr-app
- redirect README to Screenly/charlie-hr-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 c9b9859)

Here are some key observations to aid the review process:

🎫 Ticket compliance analysis ✅

822 - PR Code Verified

Compliant requirements:

  • Migrate edge-apps/charlie-hr-app from the Playground monorepo to its own standalone repository.
  • The standalone repository should be Screenly/charlie-hr-app.

Requires further human verification:

  • Verify that the standalone Screenly/charlie-hr-app repository contains the migrated source, manifests, and documentation needed to replace the removed monorepo files.
⏱️ 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 c9b9859

@github-actions

Copy link
Copy Markdown

PR Code Suggestions ✨

No code suggestions found for the PR.

@514sid 514sid merged commit d913fdb into master Jun 29, 2026
6 checks passed
@514sid 514sid deleted the chore/remove-charlie-hr-app branch June 29, 2026 08:18
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/charlie-hr-app to standalone repo (Screenly/charlie-hr-app)

2 participants