Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Import legacy products #12

Merged
merged 179 commits into from
May 30, 2021
Merged

Import legacy products #12

merged 179 commits into from
May 30, 2021

Conversation

milesmcc
Copy link
Collaborator

Type of pull request: product additions

Related issues: n/a


This PR brings in all of the nearly-correct product assessments from the old PrivacySpy site. All of the policies have some outstanding issue that prevented them from being merged in the original batch. (Note: all products that had no graded questions were removed.)

@milesmcc milesmcc self-assigned this Jun 24, 2020
@keywordlabeler keywordlabeler bot added the product Related to a product on PrivacySpy label Jun 24, 2020
@doamatto
Copy link
Collaborator

Fixed the merge conflicts, but it appears the check is getting stuck on installing fsevents (which it can't; the check uses Linux, not macOS). It's possible npm i --no-optional might resolve the problem, but it seems to be a non-issue locally (even after cache clearning, rebuilding the lockfile, et cetera).

@doamatto
Copy link
Collaborator

@milesmcc You think you might know the remedy to this issue?

@milesmcc
Copy link
Collaborator Author

Taking a look at this now. Out of curiosity, what was the need for modifying package.json and adding those packages?

@milesmcc
Copy link
Collaborator Author

Tests now work @doamatto. Still a few icons that need smaller icons, though :(

@doamatto
Copy link
Collaborator

Taking a look at this now. Out of curiosity, what was the need for modifying package.json and adding those packages?

When I was trying to run npm test or gulp, it would throw errors saying that a module was not found with that package name. Installing the packages seemed to remove the issue and continue to build. It's possible it was just a per-case scenario, but I'm not 100% sure.

@doamatto
Copy link
Collaborator

Tests now work @doamatto. Still a few icons that need smaller icons, though :(

Alright, I'll manually shrink some down in a titbit to fit the criteria. Maybe we should find a way to tell ImgBot compress down to 30kB, if possible.

@doamatto
Copy link
Collaborator

@milesmcc I believe we are good to go 😃

@milesmcc
Copy link
Collaborator Author

Great. There are a few product labels that need improvement (ESPN, for example) but I can deal with that.

@doamatto
Copy link
Collaborator

doamatto commented May 23, 2021

Hey @milesmcc; wanted to see if you need/wanted some help with it since it's been a hot while.

@milesmcc
Copy link
Collaborator Author

Honestly, sure. Would you be able to go over the descriptions for the products and make sure that they aren't super weird? For example, right now the description for ESPN is quite corporate and sounds like it was copy-pasted right off their website. In general, we want descriptions to be informative, neutral, and useful to your average visitor. The current descriptions on the site provide a good example (in terms of both content and length).

@doamatto
Copy link
Collaborator

Honestly, sure. Would you be able to go over the descriptions for the products and make sure that they aren't super weird? For example, right now the description for ESPN is quite corporate and sounds like it was copy-pasted right off their website. In general, we want descriptions to be informative, neutral, and useful to your average visitor. The current descriptions on the site provide a good example (in terms of both content and length).

No problem; I'll try to get it done up by this Friday or so.

@milesmcc
Copy link
Collaborator Author

Cool, thanks! And no rush (obviously :)).

doamatto added 3 commits May 25, 2021 18:46
This commit simplified several products to remove wordiness and extra
information, cut down on biased terminology, reduce the usage of vague
terms like "popular", properly define the origins of products, as well
as improve products that had vague descriptions.
@doamatto
Copy link
Collaborator

@milesmcc Seems it went faster than I thought it would haha. You can check d72c02d for all the changes, as well as the git message for a more in-depth of what I did. I think it might be a smart idea to establish in the contribution guidelines that people should use non-vague or "biased" terminology when doing descriptions and contributing other content.

@doamatto
Copy link
Collaborator

@milesmcc Just checking to see when you reckon you might have a chance to look this PR over since it should be good to go (🤞)

@milesmcc milesmcc marked this pull request as ready for review May 30, 2021 18:30
@milesmcc milesmcc merged commit 2f4f35b into Politiwatch:master May 30, 2021
@milesmcc milesmcc mentioned this pull request Jun 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
icon How products look on PrivacySpy product Related to a product on PrivacySpy size/XXL
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants