add exclusion setting for domains ( related issue #353 )

[KMohZaid]

Problem in this PR is : I am "noob" and i don't know how to use i18n translation . so we have to do it.

[KMohZaid]

related issue : https://github.com/ClearURLs/Addon/issues/353

(wow sonarcloud, i didnt know they have this intregation for free)

[KMohZaid]

lol, i knew i should do something to keep 1 return url statement but lazy to do and it was feeling complex if i does

and sad, i forgot to remove test console.log

[KMohZaid]

preview

[Beagle]

@KMohZaid +1 to have this merged in and released. I would love to finally have this feature in ClearURLs. It would make my life so much better.

[KMohZaid]

Lol I didn't notice a mistake

In regex explanation, after m there was ? which i think i accidentally removed

[KMohZaid]

Quality Gate passed

Issues 5 New issues 0 Accepted issues

Measures 0 Security Hotspots 0.0% Coverage on New Code 0.0% Duplication on New Code

See analysis details on SonarCloud

fix: 5 issues reported by sonarcloud 4e31424

(https://github.com/ClearURLs/Addon/pull/383#issuecomment-2402006872)

1. using variable to store boolean and if any boolean is true, then return url
2. using for-of for list
3. we are using "storage" variable to store dictionary data not array, so made it "{}" from "[]"
4. "value" parameter in setData was string when case was "excludeDomains", still removed its console.log (not needed to console log it now so)

[sonarqubecloud[bot]]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

[KMohZaid]

Quality Gate passed

Issues 0 New issues 0 Accepted issues

Measures 0 Security Hotspots 0.0% Coverage on New Code 0.0% Duplication on New Code

See analysis details on SonarCloud

nice

[tcelestino]

This PR is a feature to solves the #387 issue.

[KMohZaid]

This PR is a feature to solves the #387 issue.

No, this is just workaround. I think we should check what are tracking parameters for Google sheets domain and add rules for them. This feature will trust domain and allows unfiltered url loading

[Neustradamus]

@KMohZaid: Can you create a PR on Gitlab too?

• https://gitlab.com/ClearURLs

If it is not merged, I think it is the moment to fork this project.

Linked to:

• https://github.com/ClearURLs/Addon/pull/372