Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added Nebula #140

Closed
wants to merge 8 commits into from
Closed

Added Nebula #140

wants to merge 8 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Sep 1, 2022

Type of pull request: product addition


added Nebula and Nebula logo

Note: Standard is the owner of Nebula and their policies apply to all their products (including Nebula). This is why Standard's privacy policy is also referenced.

david added 5 commits September 1, 2022 10:04
Note: Standard is the owner of Nebula and their policies apply to all their products (including Nebula). This is why Standard's privacy policy is also referenced.
Note: Standard is the owner of Nebula and their policies apply to all their products (including Nebula). This is why Standard's privacy policy is also referenced.
Will be added back
Note: Standard is the owner of Nebula and their policies apply to all their products (including Nebula). This is why Standard's privacy policy is also referenced.
@privacyspy-bot
Copy link

privacyspy-bot bot commented Sep 1, 2022

Thanks for submitting this pull request. @milesmcc has been assigned to review these changes, provide feedback, and determine next steps.

If you haven't already, please ensure your changes pass all the automated tests. Look in the "Checks" box below and "Files changed" tab to see test results.

To learn about the PrivacySpy contribution process, check out the contribution guide.

Note to maintainers: if this pull request passes all tests and code reviews, it should be squashed and merged.

@privacyspy-bot privacyspy-bot bot added size/M icon How products look on PrivacySpy product Related to a product on PrivacySpy labels Sep 1, 2022
Copy link
Collaborator

@doamatto doamatto left a comment

Choose a reason for hiding this comment

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

Few changes to be made, alongside making sure the checks (GitHub Actions) passes fine.

products/nebula.toml Outdated Show resolved Hide resolved
products/nebula.toml Show resolved Hide resolved
products/nebula.toml Show resolved Hide resolved
products/nebula.toml Show resolved Hide resolved
products/nebula.toml Outdated Show resolved Hide resolved
@ghost
Copy link
Author

ghost commented Sep 10, 2022

Gone through and updated everything, should be good now 👍

Copy link
Collaborator

@doamatto doamatto left a comment

Choose a reason for hiding this comment

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

It seems the girth of this PR is from the standard.tv privacy policy that seems to no longer be available. This is, naturally, a problem that should be fixed.

description = "Nebula is a creator-focused video streaming platform owned by Standard.tv."
slug = "nebula"
hostnames = [ "nebula.app" ]
sources = [ "https://nebula.app/privacy", "https://standard.tv/privacy/" ]
Copy link
Collaborator

Choose a reason for hiding this comment

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

https://standard.tv/privacy/ is returning a 404 for me. Is it possible the policy was removed from there recently?

@ghost ghost closed this by deleting the head repository Dec 13, 2022
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
icon How products look on PrivacySpy product Related to a product on PrivacySpy size/M
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants