Skip to content

chore: migrate powerbi to standalone repo#863

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

chore: migrate powerbi to standalone repo#863
514sid merged 2 commits into
masterfrom
chore/remove-powerbi

Conversation

@nicomiguelino

@nicomiguelino nicomiguelino commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

User description

Removes source files from edge-apps/powerbi and redirects to Screenly/powerbi-app. Closes #833.


PR Type

Other, Documentation


Description

  • Remove monorepo Power BI app

  • Redirect docs to standalone repository

  • Drop obsolete manifests and tooling


Diagram Walkthrough

flowchart LR
  monorepo["Playground Power BI app"]
  removed["Source and manifests removed"]
  redirect["README redirects users"]
  standalone["Screenly/powerbi-app"]
  monorepo -- "migrated out" --> removed
  removed -- "replaced by" --> redirect
  redirect -- "points to" --> standalone
Loading

File Walkthrough

Relevant files
Miscellaneous
9 files
globals.d.ts
Remove Power BI Screenly settings types                                   
+0/-12   
main.ts
Remove Power BI application entrypoint                                     
+0/-19   
services.lib.ts
Remove Power BI service helper library                                     
+0/-82   
services.ts
Remove Power BI embedding service logic                                   
+0/-190 
services.types.ts
Remove Power BI service type definitions                                 
+0/-12   
utils.lib.ts
Remove Power BI utility constants                                               
+0/-8     
utils.ts
Remove Power BI utility functions                                               
+0/-62   
style.css
Remove Power BI app styling                                                           
+0/-76   
index.html
Remove Power BI app HTML shell                                                     
+0/-29   
Tests
2 files
services.test.ts
Remove Power BI service unit tests                                             
+0/-458 
utils.test.ts
Remove Power BI utility unit tests                                             
+0/-140 
Documentation
2 files
README.md
Update Power BI repository link                                                   
+1/-1     
README.md
Replace docs with migration notice                                             
+2/-24   
Configuration changes
4 files
.ignore
Remove Power BI ignore rules                                                         
+0/-4     
screenly.yml
Remove Power BI production manifest                                           
+0/-62   
screenly_qc.yml
Remove Power BI QC manifest                                                           
+0/-62   
tsconfig.json
Remove Power BI TypeScript configuration                                 
+0/-8     
Dependencies
1 files
package.json
Remove Power BI package configuration                                       
+0/-31   

- remove source files from edge-apps/powerbi
- redirect README to Screenly/powerbi-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 07109c7)

Here are some key observations to aid the review process:

🎫 Ticket compliance analysis ✅

833 - Fully compliant

Compliant requirements:

  • Migrate edge-apps/powerbi out of the Playground monorepo.
  • Point users to the standalone repository at Screenly/powerbi-app.
⏱️ 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 07109c7

@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
514sid
514sid previously approved these changes Jun 29, 2026
@salmanfarisvp salmanfarisvp self-requested a review June 29, 2026 09:25
salmanfarisvp
salmanfarisvp previously approved these changes Jun 29, 2026
@514sid 514sid dismissed stale reviews from salmanfarisvp and themself via 039f0f8 June 29, 2026 09:29
@514sid 514sid force-pushed the chore/remove-powerbi branch from 1fedc2f to 039f0f8 Compare June 29, 2026 09:29
@514sid 514sid merged commit 1066a1e into master Jun 29, 2026
4 checks passed
@514sid 514sid deleted the chore/remove-powerbi branch June 29, 2026 09:31
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/powerbi to standalone repo (Screenly/powerbi-app)

3 participants