Skip to content

Commit

Permalink
Updated Contribution Guide + Removed Icons
Browse files Browse the repository at this point in the history
I have updated the contribution guide to include what I believe are appropriate icon requirements. I have also removed icons that all use a single-sign on account and moved them to a different folder as I'm unsure what you would prefer to do with these.
  • Loading branch information
RobertHarnett committed Jun 3, 2022
1 parent a9365ed commit aa869a8
Show file tree
Hide file tree
Showing 36 changed files with 17 additions and 5 deletions.
19 changes: 15 additions & 4 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
@@ -1,13 +1,23 @@
# Contribution Guide

## Translations
## Translations 💬

Translations are now managed on Crowdin. Go to the [Authenticator Pro Crowdin project](https://crwd.in/authenticator-pro) to contribute. If your language is not available, please contact me and I will add it.
<br></br>

## Icons
## Icons ⏺️

If you'd like to contribute some icons, check the [missing icons list](./extra/missing_icons.txt) for some that might need adding. They may already exist, so check first!
If you'd like to contribute some icons, first check if there's [any open issues from user requests](https://github.com/jamie-mh/AuthenticatorPro/issues?q=is%3Aopen+is%3Aissue+label%3Aenhancement). Alternatively take a look at the [missing_icons.txt file](./extra/missing_icons.txt). Keep in mind some the the services listed here may already have an icon or not meet the criteria to get one so make sure to check first.

### Icon criteria:
Not every service needs an icon. To prevent the app having hundreds of icons from obscure and rarely used platforms, we limit what icons can be added. If a service doesn't meet the criteria we encourage the use of custom icons from within the app.

- Platforms that use a 'Single Sign-On' should have the icon added for the sign-on account and not for the individual platforms. Eg: instead of a YouTube icon a Google icon would be needed, or instead of a Photoshop icon an Adobe icon should be used.
- Web based platforms should be within [Similarweb's](https://www.similarweb.com) top 200,000 global rank. [Simply search for the site and see for yourself](https://www.similarweb.com).
- Mobile platforms should have at least 100k+ downloads on the [Google Play Store](https://play.google.com/).
- If the service is not web based or on the Play Store it will have to be reviewed individually, in which case it's best to just [submit a request as a issue.](https://github.com/jamie-mh/AuthenticatorPro/issues/new?assignees=&labels=enhancement&template=icon_request.md&title=)

### How to add an icon:
To add an icon to the project the procedure is as follows:

* Fork the repo
Expand Down Expand Up @@ -35,7 +45,8 @@ To add an icon to the project the procedure is as follows:
* Commit your changes

* Create a pull request with your changes
<br></br>

## Code / Features
## Code / Features ⚙️

Before submitting any code, please open an issue to discuss if the feature is relevant.
3 changes: 2 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -41,6 +41,7 @@ Authenticator Pro is currently only available on the F-Droid client through the
[Contribution Guide](https://github.com/jamie-mh/AuthenticatorPro/blob/master/CONTRIBUTING.md)

[Backup File Format](https://github.com/jamie-mh/AuthenticatorPro/blob/master/doc/BACKUP_FORMAT.md)
<br/><br/>

### Importing from other apps:

Expand Down Expand Up @@ -94,7 +95,7 @@ Authenticator Pro has a beautiful material design inspired look in either light
![Screenshot 8](./doc/screenshot8.png)
<br/><br/>

## Wear OS Companion
### Wear OS Companion

![Screenshot 1](./doc/wearos_screenshot1.png)
![Screenshot 2](./doc/wearos_screenshot2.png)
Expand Down
File renamed without changes
File renamed without changes
File renamed without changes
File renamed without changes
File renamed without changes
File renamed without changes
File renamed without changes
File renamed without changes
File renamed without changes
File renamed without changes
File renamed without changes
File renamed without changes
File renamed without changes
File renamed without changes
File renamed without changes
File renamed without changes
File renamed without changes
File renamed without changes
File renamed without changes
File renamed without changes
File renamed without changes
Binary file added extraicons/protonmail.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file added extraicons/protonvpn.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
File renamed without changes
File renamed without changes
File renamed without changes
File renamed without changes
File renamed without changes
File renamed without changes
File renamed without changes
File renamed without changes
Binary file added icons/proton.png
Binary file removed icons/protonmail.png
Diff not rendered.
Binary file removed icons/protonvpn.png
Diff not rendered.

0 comments on commit aa869a8

Please sign in to comment.