Skip to content

Search for pronouns in field values, names and the bio#72

Draft
nachtjasmin wants to merge 8 commits into
mainfrom
61-search-for-pronounspage-link-in-all-fields
Draft

Search for pronouns in field values, names and the bio#72
nachtjasmin wants to merge 8 commits into
mainfrom
61-search-for-pronounspage-link-in-all-fields

Conversation

@nachtjasmin

@nachtjasmin nachtjasmin commented Jul 25, 2023

Copy link
Copy Markdown
Contributor

The old approach was refactored. With this PR we are applying all searches to each individual field, including their name and value. This approach is optimized for speed in that regard, that we do the API calls as the last step.

I also embedded a list of all known pronoun.page pronouns, which can be generated using node scripts/generateKnownPronouns.mjs. The current state is commited to the repository and could be regenerated at any time.

A lot of the cases mentioned in #63 are going to be closed with this PR.

@nachtjasmin nachtjasmin linked an issue Jul 25, 2023 that may be closed by this pull request
@nachtjasmin nachtjasmin requested a review from ItsVipra July 25, 2023 16:30
@ItsVipra

ItsVipra commented Aug 1, 2023

Copy link
Copy Markdown
Owner

With all the new pronouns from all languages we get false positives, such as "I" for https://botsin.space/@lesbianBot.

Also logging could be a bit improved - the addon doesn't output any info after fetching a status

@nachtjasmin nachtjasmin force-pushed the 61-search-for-pronounspage-link-in-all-fields branch from 87aef3e to abcbadb Compare August 1, 2023 13:53
nachtjasmin and others added 7 commits August 14, 2023 13:57
The old approach was refactored. With this commit we are applying all
searches to each individual field, including their name and value. This
approach is optimized for speed in that regard, that we do the API calls
as the last step.
too many false positives for comma
@nachtjasmin nachtjasmin force-pushed the 61-search-for-pronounspage-link-in-all-fields branch from 71bb62e to 907405d Compare August 14, 2023 12:13
@nachtjasmin

Copy link
Copy Markdown
Contributor Author

Although it looks fine to me, I won't merge it unless I've fixed the following problem:

Error [ERR_MODULE_NOT_FOUND]: Cannot find package 'webextension-polyfill' imported from /home/jasmin/repos/protoots/src/libs/settings.js
    at new NodeError (node:internal/errors:399:5)
    at packageResolve (node:internal/modules/esm/resolve:889:9)
    at moduleResolve (node:internal/modules/esm/resolve:938:20)
    at defaultResolve (node:internal/modules/esm/resolve:1153:11)
    at nextResolve (node:internal/modules/esm/loader:163:28)
    at ESMLoader.resolve (node:internal/modules/esm/loader:838:30)
    at ESMLoader.getModuleJob (node:internal/modules/esm/loader:424:18)
    at ModuleWrap.<anonymous> (node:internal/modules/esm/module_job:77:40)
    at link (node:internal/modules/esm/module_job:76:36)

@nachtjasmin nachtjasmin marked this pull request as draft August 24, 2023 11:01
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.

Search for pronouns.page link in all fields

2 participants