Updates to OSU HUB Form, Update OSU logo, and other small changes #46

Merged
Toshaan Bharvani merged 8 commits from maxschmidt/openpower.foundation:master into master 2 years ago
There is no content yet.

@toshywoshy any chance we can get this approved soon?

@toshywoshy any chance we can get this approved soon?
Lance Albertson requested changes 2 years ago
Lance Albertson left a comment
  • Please use optipng to optmize the png so that it's smaller. It's rather large for a website
  • Please remove any trailing white spaces
  • Update to current master branch
- [ ] Please use `optipng` to optmize the png so that it's smaller. It's rather large for a website - [ ] Please remove any trailing white spaces - [ ] Update to current master branch
  • Please use optipng to optmize the png so that it's smaller. It's rather large for a website

We use HugoIO's image conversion tool, hugo-extended has this feature and we can therefor convert any image to webp and optimize it scaling and ratios according to the page.

  • Please remove any trailing white spaces

Yes, please, I should work out a lint checker.

  • Update to current master branch

Yes, sorry, I didn't see this PR early enough so rebase to latest and I can approve to master

> - [ ] Please use `optipng` to optmize the png so that it's smaller. It's rather large for a website We use HugoIO's image conversion tool, ```hugo-extended``` has this feature and we can therefor convert any image to webp and optimize it scaling and ratios according to the page. > - [ ] Please remove any trailing white spaces Yes, please, I should work out a lint checker. > - [ ] Update to current master branch Yes, sorry, I didn't see this PR early enough so rebase to latest and I can approve to master

Ideally a rebase instead of a merge when updating.

Ideally a rebase instead of a merge when updating.
Maximillian Schmidt force-pushed master from 8d41de3079 to 00bda9b212 2 years ago
Maximillian Schmidt requested review from Lance Albertson 2 years ago
Maximillian Schmidt changed title from Updates to OSU HUB Form and other small changes to Updates to OSU HUB Form, Add OSU HUB Blog, and other small changes 2 years ago

Please create a seperate PR for the blog.
Without the blog, I can move to merge.

Please create a seperate PR for the blog. Without the blog, I can move to merge.
Maximillian Schmidt force-pushed master from db88fc5d68 to 29f3e2a774 2 years ago
Maximillian Schmidt changed title from Updates to OSU HUB Form, Add OSU HUB Blog, and other small changes to Updates to OSU HUB Form, Update OSU logo, and other small changes 2 years ago

Well that wasn't right. Working on it.

Well that wasn't right. Working on it.
Maximillian Schmidt force-pushed master from 357206bb3f to 0c77a733e3 2 years ago
Toshaan Bharvani approved these changes 2 years ago
Toshaan Bharvani merged commit 9e4f5c909e into master 2 years ago

Reviewers

Lance Albertson was requested for review 2 years ago
Toshaan Bharvani approved these changes 2 years ago
The pull request has been merged as 9e4f5c909e.
Sign in to join this conversation.
No Milestone
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: website/openpower.foundation#46
Loading…
There is no content yet.