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

Add Product: ID.me (online identity verification service) #187

Merged
merged 12 commits into from
Sep 9, 2024

Conversation

opile8
Copy link
Contributor

@opile8 opile8 commented Aug 20, 2024

Type of pull request: product addition / product edit / bug fix / other...
product addition: ID.me

  • included icon and toml, used other products and rubric as guides

Related issues: list the related issues here (if applicable)


Provide more information about your pull request here. For help, see our contribution guide: http://privacyspy.org/contribute/

First draft of product toml
Add biometric policy citations and update sources.
correct typo
cleanup, validate TOML format
Copy link
Collaborator

@doamatto doamatto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few things that need tweaking. There was also the ID.me Rx section near the bottom that I forget if was included in this. Overall, good start ^_^

products/id-me.toml Outdated Show resolved Hide resolved
products/id-me.toml Outdated Show resolved Hide resolved
products/id-me.toml Show resolved Hide resolved
products/id-me.toml Outdated Show resolved Hide resolved
products/id-me.toml Show resolved Hide resolved
products/id-me.toml Outdated Show resolved Hide resolved
products/id-me.toml Outdated Show resolved Hide resolved
products/id-me.toml Outdated Show resolved Hide resolved
products/id-me.toml Show resolved Hide resolved
products/id-me.toml Outdated Show resolved Hide resolved
image failed verification
will upload png that is mostly square
image validation failed, should be more squared
@opile8 opile8 requested a review from doamatto September 9, 2024 01:00
Copy link
Collaborator

@doamatto doamatto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good now. Thanks !

@doamatto doamatto merged commit 57a13b3 into Politiwatch:master Sep 9, 2024
5 checks passed
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.

2 participants