search

threshold-network / website

1 stars 10 forks source link

Assorted housekeeping #61

Closed mswilkison closed 1 year ago

mswilkison commented 1 year ago

This PR makes assorted updates and corrections to the website.

liz-shinn commented 1 year ago

Nice! Thanks Maclane!

mswilkison commented 1 year ago

Rebased this PR to squash the commit log and remove any dependency changes. Not sure why the deploy preview check is failing since it's just content changes?

georgeweiler commented 1 year ago

The changes look fine. I pinged @michalinacienciala to check on the pipeline. Thanks @mswilkison !

georgeweiler commented 1 year ago

@mswilkison one guess is that it's because your commit is not signed/verified?

mswilkison commented 1 year ago

@mswilkison one guess is that it's because your commit is not signed/verified?

Signed and force pushed, but no dice.

mswilkison commented 1 year ago

:/ I rebased and resolved conflicts with the latest PR that was just merged, but still failing.

This PR removes links to several renBTC exposed pools so it'd be good if we can get it resolved and merged in, otherwise we are quasi-endorsing pools that are going to get people rekt.

r-czajkowski commented 1 year ago

:/ I rebased and resolved conflicts with the latest PR that was just merged, but still failing.

Maybe because it's a PR from your fork repo 🤔 @mswilkison do you mind if I cherry-pick your commit and open a separate PR? Will probably be easier than investigating why it fails.

mswilkison commented 1 year ago

:/ I rebased and resolved conflicts with the latest PR that was just merged, but still failing.

Maybe because it's a PR from your fork repo 🤔 @mswilkison do you mind if I cherry-pick your commit and open a separate PR? Will probably be easier than investigating why it fails.

Sure, that's fine with me

r-czajkowski commented 1 year ago

Created a separate PR due to failing build see #65.