0.4.2

[rougetimelord]

This is a long-lived branch, intended to aggregate all changes planned for version 0.4.2.

Pull requests should be made against this branch. Merges to this branch should be done via a merge commit. When ready, this branch should be merged to main via a squash commit.

Changelog

• [chore] bump version string
• [fix] change condition for checking legacy verified DB if user used blue features
  • PR: #266
• [doc] add all properties to the BlueBlockerUser type definition
  • PR: #268
• [fix] refix skipping deleted tweets
  • PR: #269
• [fix] remove reference to chrome.runtime.restartAfterDelay
  • Removes a firefox extension API incompatibility
  • PR: #271
• [feat] add a way for external extensions to register themselves as an integration
  • PR: #274
• [chore] change firefox release workflow
  • Please work this time! 😭
  • H/T: @kheina, PR: #277
• [feat] add blocking users for words/characters in their display name
  • H/T @MaxIsJoe, PR: #275
  • Closes #114
• [feat] add ability to skip blocking for blue users
  • H/T @MaxIsJoe, PR: #278
  • Closes #166
• [fix] allow extension to run on on x.com
  • H/T @kheina, PR: #282
  • Closes #280
• [chore] run prettier against all code

Deployment Checklist

1. [x] merge all pull requests to llb
2. [x] ensure src/manifest.ts and package.json have the correct version number
3. [ ] use makefile to generate zips (make chrome, make firefox)
   • [x] chrome should be tested with npm run build
   • [ ] test firefox locally using zip
4. [ ] merge llb to main

[kheina]

I would like to do a full review before we merge anything this large, I'll do that tonight. I also have thoughts on what to do regarding the emoji/words regex which I'll add when I get to that part of the pr.

tonight being ~4-5 hours from now, I'm at work currently

[kheina]

this is what I meant by having a "compiled" config for running on vs saving. this way we're not saving extra data and only compiling once per request https://github.com/kheina-com/Blue-Blocker/pull/283

[rougetimelord]

Going to start testing! Can confirm that with the fixes the extension can queue and block people again

[rougetimelord]

Problems:

• [x] Empty disallowed words list seems to spam block everything
• [x] Disallowed words blocking logic does not respect the pause collection setting not an issue, i think...
• [x] TypeError in popup script, line 72: cannot read property of undefined " join"

[MaxIsJoe]

Problems:

• [ ] Empty disallowed words list seems to spam block everything

I added this check specifically to address this issue in my original PR.

[rougetimelord]

Problems:

• [ ] Empty disallowed words list seems to spam block everything

I added this check specifically to address this issue in my original PR.

Yea, but then we switched to precompiling the regex, so I'll futz around with it

[rougetimelord]

Ok that's all the bugs I've found so far squashed. Going to keep testing for tonight to see if something pops up, but I think we're close to being able to merge.

[kheina]

I'll do another review and some testing tonight after work. hopefully it can go out tonight

[kheina]

I'll do another review and some testing tonight after work. hopefully it can go out tonight

I'll probably also just push a bunch of styling changes/fixes since I'm pretty particular about the appearance of the popup menu

[rougetimelord]

I'll do another review and some testing tonight after work. hopefully it can go out tonight

I'll probably also just push a bunch of styling changes/fixes since I'm pretty particular about the appearance of the popup menu

Go ham! I think having the setting be hidden behind a checkbox if not enabled would make a lot of sense in terms of not cluttering up the interface, kinda like what the skip threshold setting does

[kheina]

I'll do another review and some testing tonight after work. hopefully it can go out tonight

I'll probably also just push a bunch of styling changes/fixes since I'm pretty particular about the appearance of the popup menu

Go ham! I think having the setting be hidden behind a checkbox if not enabled would make a lot of sense in terms of not cluttering up the interface, kinda like what the skip threshold setting does

I went ham

[rougetimelord]

One approval pls 👉🏻👈🏻