Skip to content

Conversation

@fleupold
Copy link
Contributor

Copying relevant instructions from slack here, so that internal and external contributors can make suggestions (reducing our manual work).

@fleupold fleupold requested a review from anxolin December 15, 2022 09:21
Copy link
Contributor

@anxolin anxolin left a comment

Choose a reason for hiding this comment

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

Thanks for adding this into the README 🚀

README.md Outdated

In order to add a new token to the CoW Swap default token list follow these steps

- Find an icon with reasonable quality/size (~256px, <100kB).
Copy link
Contributor

Choose a reason for hiding this comment

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

Why not exactly 256x256? I think most of the current images in the list (if not all have that size exactly).

@fairlighteth might have some extra-requirements (or different requirements. What is a good process here?

- Make sure you bump the version using SemVersions
- Create a PR with these changes

[^1]: To resize/convert your image [ImageMagick](https://formulae.brew.sh/formula/imagemagick) installed you can simply run `convert ~/Downloads/img.png -resize 256 src/public/images/<chainId>/<address>.png`
Copy link
Contributor

Choose a reason for hiding this comment

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

Does this optimize like https://tinypng.com do?
cc/ @fairlighteth

Copy link
Collaborator

@alfetopito alfetopito left a comment

Choose a reason for hiding this comment

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

LGTM

@alfetopito
Copy link
Collaborator

Merging as is to have some instructions in the readme.
We can re-iterate if needed, but this is better than no instructions and people not knowing what to do :)

@alfetopito alfetopito merged commit d04adb4 into main Jan 27, 2023
@alfetopito alfetopito deleted the readme branch January 27, 2023 11:30
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.

4 participants