Skip to content

Conversation

@Julusian
Copy link
Member

@Julusian Julusian commented Oct 6, 2025

About the Contributor

This pull request is posted on behalf of Superfly

Type of Contribution

This is a: Feature / Code improvement

New Behavior

As part of #1450 and following #1534, this removes the remnants of organization support from Sofie.

Organizations were originally added to track ownership of resources/documents within an installation with multiple studios needing different 'owners'. As Sofie is being limited to one Studio, Organizations do nothing for us.

A lot of this was pretty dead code, very few places were setting or checking the organizationId field on documents

Testing

  • I have added one or more unit tests for this PR
  • I have updated the relevant unit tests
  • No unit test changes are needed for this PR

Affected areas

Time Frame

Other Information

Status

  • PR is ready to be reviewed.
  • The functionality has been tested by the author.
  • Relevant unit tests has been added / updated.
  • Relevant documentation (code comments, system documentation) has been added / updated.

@Julusian Julusian requested a review from a team as a code owner October 6, 2025 11:07
@Julusian Julusian added Contribution from SuperFly.tv Contributions sponsored by SuperFly.tv Contribution External contribution labels Oct 6, 2025
@Julusian Julusian changed the title feat: remove remnants of 'organisations' feat: remove remnants of 'organizations' Oct 6, 2025
@codecov-commenter
Copy link

codecov-commenter commented Nov 12, 2025

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

❌ Patch coverage is 31.03448% with 20 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
meteor/server/api/snapshot.ts 8.33% 11 Missing ⚠️
meteor/server/api/studio/api.ts 0.00% 3 Missing ⚠️
meteor/server/publications/organization.ts 0.00% 2 Missing ⚠️
meteor/server/api/showStyles.ts 66.66% 1 Missing ⚠️
meteor/server/collections/index.ts 0.00% 1 Missing ⚠️
meteor/server/migration/databaseMigration.ts 0.00% 1 Missing ⚠️
meteor/server/security/check.ts 50.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@Julusian Julusian merged commit de8774a into Sofie-Automation:release53 Nov 12, 2025
31 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Contribution from SuperFly.tv Contributions sponsored by SuperFly.tv Contribution External contribution

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants