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 various websites to the non-global OneTrust rules #335

Merged
merged 2 commits into from
Dec 13, 2023

Conversation

Greg-21
Copy link
Contributor

@Greg-21 Greg-21 commented Dec 4, 2023

I suppose that at some point, the non-global OneTrust rules will have to go, but until then, I might as well take advantage of them and add a bunch of websites I came across while browsing the Internet.

Fixes #294, fixes #334
Resolves #154, resolves #193

Closes #71, closes #74

@Greg-21 Greg-21 requested a review from a team as a code owner December 4, 2023 23:45
@abhishekmadan30
Copy link
Contributor

Would it possible for you to resubmit the patch without reordering the existing rules

@Greg-21
Copy link
Contributor Author

Greg-21 commented Dec 6, 2023

Here's an alternative commit with the current domain order preserved: link

Is that format better? If so, I can always modify the commit in this pull request as well.

@Greg-21 Greg-21 force-pushed the non-global-onetrust-rules branch from 6cd180c to 1f4bff0 Compare December 10, 2023 22:31
@abhishekmadan30
Copy link
Contributor

Could you please reformat the commit like the linked commit you had provided

@Greg-21
Copy link
Contributor Author

Greg-21 commented Dec 12, 2023

Yes, I will do that.

@Greg-21 Greg-21 force-pushed the non-global-onetrust-rules branch from 1f4bff0 to ce0c602 Compare December 12, 2023 21:06
Copy link
Contributor

@abhishekmadan30 abhishekmadan30 left a comment

Choose a reason for hiding this comment

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

Nice, Thanks :)

@abhishekmadan30 abhishekmadan30 merged commit 70094ab into mozilla:main Dec 13, 2023
1 check passed
@Greg-21 Greg-21 deleted the non-global-onetrust-rules branch December 13, 2023 22:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants