Skip to content

Add support for custom rules#250

Open
kgmt0 wants to merge 2 commits into
ClearURLs:masterfrom
kgmt0:custom-rules
Open

Add support for custom rules#250
kgmt0 wants to merge 2 commits into
ClearURLs:masterfrom
kgmt0:custom-rules

Conversation

@kgmt0

@kgmt0 kgmt0 commented Aug 25, 2022

Copy link
Copy Markdown

This adds a textarea in the settings page that can be used to enter custom rules in the same format as data.json. These rules are applied on top of the ones from data.json.

kgmt0 added 2 commits August 26, 2022 03:38
This adds a textarea in the settings page that can be used to enter
custom rules in the same format as data.json. These rules are applied on
top of the ones from data.json.
start.getKeys() was using a for...in loop to iterate over fields in the
given object, meaning that methods names (e.g. getOrDefault()) get
included in the iteration as well. This causes erroneous entries to be
added the global list of providers. Fix this by using Object.keys() to
extract field names and iterate over that instead.
@sonarqubecloud

Copy link
Copy Markdown

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 2 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@TheCrazyMax

Copy link
Copy Markdown

That's exactly what I was looking for in the extension options, thinking this was a base feature.
Is there any plan to merge this ? :)

@inson1

inson1 commented Dec 15, 2023

Copy link
Copy Markdown

@KevinRoebert ?

@WolfganP

Copy link
Copy Markdown

Good idea. It will also help to submit improved rules to the core set, as advanced users may test the rules easier.

@Acelya-9028

Copy link
Copy Markdown

Please merge it :s

@mtsz-pl

mtsz-pl commented Nov 18, 2024

Copy link
Copy Markdown

@KevinRoebert this PR exists for more than two years now, could you please merge it or decline, so that we all know how things are with this feature?

@rugabunda

Copy link
Copy Markdown

Add this feature please

@Neustradamus

Copy link
Copy Markdown

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

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

@kgmt0

kgmt0 commented Aug 6, 2025

Copy link
Copy Markdown
Author

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

* https://gitlab.com/ClearURLs

https://gitlab.com/ClearURLs/ClearUrls/-/merge_requests/111

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants