Skip to content

Conversation

@mwelling
Copy link
Contributor

Add page and images for the OHS2020 badge.

Michael Welling added 2 commits January 16, 2020 16:23
Signed-off-by: Michael Welling <mwelling@ieee.org>
3D render for now
@mwelling
Copy link
Contributor Author

Per request on adafruit/circuitpython#2528

@mwelling mwelling requested review from ladyada and theacodes January 17, 2020 00:37
@mwelling
Copy link
Contributor Author

Sorry for taking so long I had to create the 3D renders because I don't have any board.

Copy link
Collaborator

@makermelissa makermelissa left a comment

Choose a reason for hiding this comment

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

Hi, thanks for submitting. Could you pare the features down to what's in the list on this page? https://learn.adafruit.com/how-to-add-a-new-board-to-the-circuitpython-org-website/adding-to-downloads

Thanks

Michael Welling added 2 commits January 16, 2020 16:42
Signed-off-by: Michael Welling <mwelling@ieee.org>
Signed-off-by: Michael Welling <mwelling@ieee.org>
@mwelling
Copy link
Contributor Author

@makermelissa Is that better?

@makermelissa
Copy link
Collaborator

Yes, thank you. I have one other small change is to request so that the images look their best on the site. Would you be able to resize the large and small images to 780x600 and 293x225? If not, I could do it in a separate PR. Thanks.

@mwelling
Copy link
Contributor Author

Sure lemme take a look.

@mwelling
Copy link
Contributor Author

Better?

Copy link
Collaborator

@makermelissa makermelissa left a comment

Choose a reason for hiding this comment

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

Yep, looks great!

@makermelissa makermelissa merged commit 37860ad into adafruit:master Jan 17, 2020
@makermelissa
Copy link
Collaborator

Thank you!

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