Extend Hardware Section #989
Loading…
x
Reference in New Issue
Block a user
No description provided.
Delete Branch "ghbjklhv/patch-6"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Description: Add Extended Hardware Section based on #616 and #904
What was changed: I recommend looking through the preview:
https://deploy-preview-989--privacytools-io.netlify.com/classic/#laptops
Why isn't ___ blank listed?: Only privacy and free software supporting projects are listed.
Basically all recommendations have had to some degree of audit. If you think I skipped something feel free to recommend changes in the comments!
Edits from maintainers are welcome!
Deploy preview for privacytools-io ready!
Built with commit
9f55209641
https://deploy-preview-989--privacytools-io.netlify.com
Something about this line is broken in the preview.
Sincerest apologies, I commented this out for /old and must've forgotten to fix/un-comment it. You may want to look into this as well: /diff-88e96f1b7cc4c9bc9fa893113a8e0aacR38
I'm not certain what the issue is.
To verify that Key Manager's is indeed the problem, I'll comment it out.
If this is the issue, I'm not certain how it is different from the other cards.
Removing this line seems to enable it to load properly:
(<a href='https://github.com/Nitrokey/nitrokey-pro-firmware/issues/48'>more info</a>)
I will see if I can figure out why this line breaks it.
https://
must be removed for some reason fixes in latest commit:9f55209641
@JonahAragon
@ghbjklhv do you not test your changes locally with
bundle exec jekyll serve
before pushing? lolI'll take a look at these changes in a bit.
Edit: Oh I see you're making these changes on GitHub.com, that seems rather tedious 😝
@JonahAragon Yeah, I use the online editor now.
My computer always shows errors when compiling:
Sorry if my PR seems a bit messy. :P
I'll see if I can fix my Jekyll.....
@JonahAragon Are ya'll just waiting for #904 to clear up.
Or is something wrong with this post.
Hasn't been reviewed in over a month.
I really like this PR. I just need the SeaBIOS image fixed and then I'll approve this.
Long-term I also want to stop using
card.html
entirely on the site so we should switch everything tocardv2.html
, but if you don't want to do that now we can make that change later and I'll merge this in the meantime.@ -0,0 +10,4 @@
{% include card.html color="primary"
title="SeaBIOS"
image="/assets/img/tools/seabios.png"
This image seems to be broken?
@ -0,0 +10,4 @@
{% include card.html color="primary"
title="SeaBIOS"
image="/assets/img/tools/seabios.png"
Yes, Seabios doesn't seem to have a logo.
If it is required, they have a Wiki logo: https://www.coreboot.org/images/a/ae/Chip_tools.png
Github blocked my old account but edits from maintainers should be turned on.
Or, you can add if after merging.
Would it be possible for you to start a discussion thread for some of these on the forum?
Some of the products like the Librem 5 and EOMA68 have new status updates.[1][2]
@JonahAragon @Mikaela
Hardware is not my cup of tea, so I prefer to not get involved with this PR.
@privacytoolsIO/editorial please see the previous comment.