-
-
Notifications
You must be signed in to change notification settings - Fork 78.8k
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
add a thanks page to our doc #28744
base: main
Are you sure you want to change the base?
add a thanks page to our doc #28744
Conversation
5144138
to
2161ae9
Compare
Changed the theming a bit and did some copy changes. |
041e4bc
to
b8f5029
Compare
f9c564a
to
08ac306
Compare
Hi @znarf do you know when this endpoint: https://opencollective.com/bootstrap/members/all.json?TierId=7192 will have CORS too ? |
fabb694
to
a3c807b
Compare
@Johann-S I now recommend to start using the GraphQL API v2 for everything. https://medium.com/open-collective/open-collective-graphql-api-preview-3b42ed1d55ff If what you need is a REST endpoint with CORS, let me know exactly the required data and I'll try to make that available. |
Hi @znarf, Thanks for letting me know that 👍 but we need a REST endpoint with CORS, we don't have a backend server which will fetch the OC data for us 😟 I would love using your GraphQL but it would expose our API key publicly |
Like I said:
What the endpoint should be and which data should it returns? And on your side, which fields are you reading from? |
The end point should be this one: https://opencollective.com/bootstrap/members/all.json with CORS and should accept a The data we need for our Backers and Sponsors:
|
What do you think of that? https://rest.opencollective.com/v2/bootstrap/orders/incoming/active It's featuring all active subscriptions, has CORS and all the fields you need. Only |
Thanks @znarf ! That's awesome 👍 I saw we can use this one too: https://rest.opencollective.com/v2/bootstrap/orders/incoming |
@Johann-S maybe you want then to filter the ones with |
I don't think it's usefull for us 🤔 what you provided to us is perfect 👍 |
b3f87d9
to
e434915
Compare
Pushed a commit where we use the new API (thanks @znarf) |
6453920
to
5f7f894
Compare
38bb026
to
91a20e7
Compare
b5d9034
to
abc3a67
Compare
Is this still relevant? |
Yeah, it's relevant although probably not like it is right now. We still need to add sponsors in the homepage later. |
Images automagically compressed by Calibre's image-actions ✨ Compression reduced images by 16.4%, saving 84.00 KB.
105 images did not require optimisation. Update required: Update image-actions configuration to the latest version before 1/1/21. See README for instructions. |
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.
Might need an update of the assets to migrate to v5.3 and update the markup to adopt some new helpers we have
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 one look exactly the same as the one @2x, is that fine ?
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.
Yes, it should be changed whenever this PR is finalized. Let's keep this conversation opened to not forget it :)
@@ -0,0 +1,23 @@ | |||
<section class="row g-3 g-md-5 pb-md-5 mb-5 align-items-center"> | |||
<div class="col-lg-6"> | |||
<div class="masthead-followup-icon d-inline-block mb-3" style="--bg-rgb: var(--bd-violet-rgb);"> |
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.
Don't we have any other color in here ? It looks like duplicate of the previous one
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 already use danger, info, pink, primary, violet, teal, warning, violet on this page. The remaining colors in the palette are too close to the used one (green almost like teal, cyan like info, indigo like violet) so I'm not sure how to choose another one. @mdo could help us if we try to finalize this PR.
## Sponsors | ||
|
||
{{< sponsors.inline >}} | ||
<div id="sponsorList" class="d-flex flex-wrap mx-n2 text-center"></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.
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.
Yep, you're right, it was added in the first commits from this PR in 4c7953d. I'll check how to change the rendering.
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.
Enhancement of the rendering done via d5b5c3b. Maybe there's a better solution, but this quick one should allow to resolve this thread.
If having a new homepage section to support Bootstrap is still a thing, I could extract this independent part from this PR so that it can be tackled probably more rapidly, while we take our time for the pure sponsors page to find the right technical solution. Thoughts? (/cc @mdo) |
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.
The rendering looks reaaly good, well done! Some minor changes before merging and we're good to go on my side.
{{ if .website -}} | ||
<a href="{{ .website }}" class="stretched-link text-reset" title="{{ .name }}" target="_blank" rel="nofollow noopener noreferrer"> | ||
{{ end -}} | ||
<img src="{{ printf ("/docs/%s/assets/img/services/%s") $.Site.Params.docs_version .image }}" alt="{{ .name }}" class="mh-100 mw-100"> |
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.
Since they are static images, we have an issue with dark mode, maybe add a static background to its parent or use the svg directly (this way we can color the text using var(--bs-body-color)
or something similar) ?
) | ||
} | ||
|
||
output.push(`<img src="${backerAccount.imageUrl}" alt="${backerAccount.name}" class="img-fluid d-block">`) |
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.
To have at least the backer's pseudo displayed somehow
output.push(`<img src="${backerAccount.imageUrl}" alt="${backerAccount.name}" class="img-fluid d-block">`) | |
output.push(`<img src="${backerAccount.imageUrl}" title="${backerAccount.name}" alt="${backerAccount.name}" class="img-fluid d-block">`) |
output.push( | ||
'<div class="position-relative">', | ||
' <div class="img-thumbnail mx-auto d-flex align-items-center justify-content-center overflow-hidden sponsor">', | ||
` <img class="img-fluid d-block" src="${sponsorAccount.imageUrl}" alt="${sponsorAccount.name}">`, |
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.
` <img class="img-fluid d-block" src="${sponsorAccount.imageUrl}" alt="${sponsorAccount.name}">`, | |
` <img class="img-fluid d-block" src="${sponsorAccount.imageUrl}" title="${sponsorAccount.name}" alt="${sponsorAccount.name}">`, |
) | ||
|
||
if (sponsorAccount.website) { | ||
output.push(`<a href="${sponsorAccount.website}" class="stretched-link text-reset" title="${sponsorAccount.name}" target="_blank" rel="nofollow noopener noreferrer">${sponsorAccount.name}</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.
output.push(`<a href="${sponsorAccount.website}" class="stretched-link text-reset" title="${sponsorAccount.name}" target="_blank" rel="nofollow noopener noreferrer">${sponsorAccount.name}</a>`) | |
output.push(`<a href="${sponsorAccount.website}" class="stretched-link text-reset" target="_blank" rel="nofollow noopener noreferrer">${sponsorAccount.name}</a>`) |
})() | ||
})() | ||
</script> | ||
{{< /services.inline >}} |
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.
Feels wrong to have this closing tag after the script
one
</p> | ||
</div> | ||
<div class="col-lg-6"> | ||
<img class="img-fluid mt-3 mx-auto" srcset="/docs/{{ .Site.Params.docs_version }}/assets/img/bootstrap-sponsoring.png, |
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.
Better fit with general spirit of the page in dark mode imo
<img class="img-fluid mt-3 mx-auto" srcset="/docs/{{ .Site.Params.docs_version }}/assets/img/bootstrap-sponsoring.png, | |
<img class="img-fluid mt-3 mx-auto rounded" srcset="/docs/{{ .Site.Params.docs_version }}/assets/img/bootstrap-sponsoring.png, |
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.
Shouldn't we add a section with best or last donators in here ?
markdownify
orchomp
in any of those fieldsLive previews