AndyReifman / ArseneWenger

1 stars 12 forks source link

generalized the link fixer cog and added more domains #113

Closed ingenium21 closed 2 months ago

ingenium21 commented 2 months ago

renamed the class and file name to be more general.

added a dictionary to keep track of whatever domains need to be added or removed.

rewrote methods so that they don't just focus on twitter but the domains in the dict.

AndyReifman commented 2 months ago

Looks fine, just 1 small nitpick comment

Also just a thought, do we want to strip long URLs with referrer information etc

Yes, we definitely should strip tracking information