-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
refactor: mega revamp of devs site to match interledger.org v4 #43
Conversation
✅ Deploy Preview for interledger-devs-preview ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@@ -21,11 +21,9 @@ import DevelopersLogo from "../components/DevelopersLogo.astro"; | |||
Interledger Foundation | |||
</span> | |||
</a> | |||
<div class="secondary-wrap"> | |||
<div class="sl-hidden md:sl-flex secondary-wrap"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bit of a bug fix, because the Github icon shows up in the mobile menu, and should have been part of the secondary wrap to begin with.
@@ -20,8 +20,8 @@ const currentYear = new Date().getFullYear(); | |||
<h4>Connect with us</h4> | |||
<ul> | |||
<li> | |||
<a href="/contact" aria-label="Contact form"> | |||
<svg role="img" viewBox="0 0 253 178"><title>Mail</title><path fill-rule="evenodd" d="M253 46v109c0 3 0 5-1 8l-64-64 65-53ZM23 0h207c13 0 23 10 23 23v4l-102 84c-18 15-30 15-48 0L0 27v-4C0 10 10 0 23 0Zm218 175c-3 2-7 3-11 3H23c-4 0-8-1-11-3l66-66 15 13c24 19 44 19 67 0l15-13 66 66ZM1 163c0-3-1-5-1-8V46l65 53-64 64Z" clip-rule="evenodd"/></svg> | |||
<a href="https://www.youtube.com/@InterledgerFoundation" aria-label="Interledger's Youtube channel"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Replaced the contact form link with youtube link (did that on the main site too)
@@ -24,67 +21,75 @@ import FoundationLogo from "../FoundationLogo.astro"; | |||
<li class="menu-item menu-item--level-2"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This just updates the navigation links to match the main site.
@@ -88,27 +83,28 @@ const { title, description } = Astro.props; | |||
} | |||
} | |||
|
|||
.heading--1, | |||
h1 { | |||
.heading--1 { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove styling for base heading tags for better flexibility on individual pages.
@@ -0,0 +1,135 @@ | |||
--- | |||
import BaseLayout from "./BaseLayout.astro"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wow, new blog who dis?
@@ -0,0 +1,99 @@ | |||
--- |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
dynamic page template for when pagination kicks in (yes, this is a bit of future proofing but after 12 articles, i will add in the pagination component)
@@ -1,50 +1,66 @@ | |||
--- | |||
import HomeLayout from '../layouts/HomeLayout.astro'; | |||
import BaseLayout from '../layouts/BaseLayout.astro'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can't call it HomeLayout anymore since the blog template also needs to use this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Checked it out locally - looks nice!
It's not a change introduced here, but it looks like maybe the Contact us
and Terms and Conditions
links are wrong?
<li><a href="">Contact us</a></li> |
And maybe the terms and conditions?
<li><a href="/">Terms & conditions</a></li> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What do you think about a call to action for getting involved that leads to here (on the /developer
page)? I can envision a button similar to the "Read the documentation" one on the current version (https://interledger.org/developers/) but I think any form of direct link would be good since it seems to me like this is a pretty core purpose of the page. Currently it looks like the the only path to this article is the link in "Read the Interledger specifications" (and then clicking "Get Involved"). So it seems like people would only get here by accident.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh definitely. You know what, I'll just put the button back in.
yeah, we don't seem to actually have terms and conditions at the moment. i need to chase anna about this. |
<h3>Specifications</h3> | ||
<p>Interledger specifications</p> | ||
</a> | ||
<h2 class="heading--6"><OpenPaymentsIcon /> Open Payments</h2> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Project icons make the card headings pop more
<div class="text-wrapper"> | ||
<h1 class="hero__heading heading--2">Interledger Developers Portal</h1> | ||
<p class="hero__txt">Help shape the future of interoperable payments and enable seamless exchange of value across payment networks.</p> | ||
<a class="hero__btn" href="/developers/get-involved">Get involved</a> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great idea from @BlairCurrey
@@ -61,7 +61,7 @@ const currentYear = new Date().getFullYear(); | |||
<ul class="footer-links"> | |||
<li><a href="/">Terms & conditions</a></li> | |||
<li><a href="https://www.iubenda.com/privacy-policy/88624819">Privacy policy</a></li> | |||
<li><a href="">Contact us</a></li> | |||
<li><a href="/contact">Contact us</a></li> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix missing link.
Will have to chase Anna about the terms & conditions link, this is an issue for the main site as well.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great, just a few comments I had
</a> | ||
<h2 class="heading--6"><OpenPaymentsIcon /> Open Payments</h2> | ||
<p>Open Payments is an open API standard that can be implemented by account servicing entities to facilitate interoperability in the setup and completion of payments for different use cases.</p> | ||
<a href="https://github.com/interledger/open-payments">Learn more on Github</a> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should these link to the Open Payments and Rafiki documentation sites instead?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For now, the READMEs on the respective projects have better introductory content than the documentation sites. Until we come up with good content for the home pages of those sites, we will link to the Github pages for now.
|
||
If you are most comfortable working with **React**, you can contribute to [Rafiki](https://github.com/interledger/rafiki) or [Testnet](https://github.com/interledger/testnet). | ||
|
||
If you are most comfortable working with **Angular**, you can contribute to [Testnet](https://github.com/interledger/testnet) or [Interledger Pay](https://github.com/interledger/interledger-pay). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think we have any Angular projects, all of the frontends are react-based
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This list came from @AlexLakatos though 🤔
<section> | ||
<div class="content-wrapper"> | ||
<p class="heading--4">Learn more about the various open-source projects that Interledger is working on.</p> | ||
</div> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we didn't want to include WM here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@AlexLakatos said not. But we can always add it in later if we wanted to.
Co-authored-by: Max Kurapov <[email protected]>
Bypassing branch protections because we already made the switch on the main site and nobody cares to approve this PR. |
Closes #36
This PR:
To be merged after interledger.org v4 is launched on March 11.