Remove expired sponsors from sponsors list #273
Conversation
Deploying ddev-com-front-end with
|
| Latest commit: |
2600bc7
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://db4091f3.ddev-com-front-end.pages.dev |
| Branch Preview URL: | https://20241108-remove-not-current.ddev-com-front-end.pages.dev |
| url | ||
| avatarUrl | ||
| } | ||
| user: user(login: "rfay") { |
There was a problem hiding this comment.
are you sure we want to change the user here?
There was a problem hiding this comment.
Before we had the ddev org, some people had started sponsoring the project via the rfay account. This was also accounted for previously in the code here, but the logic was different. There are still 17 people and orgs sponsoring DDEV via the rfay account.
There was a problem hiding this comment.
I compared
https://ddev.com/resources/featured-sponsors.svg
https://20241108-remove-not-current.ddev-com-front-end.pages.dev//resources/featured-sponsors.svg
It's okay to have it arranged like this?
|
I'm not clear about what part of the arrangement you're talking about, thanks for looking. |
|
I think it looks OK, Everything I've been doing here has been just prompting AI, since I don't understand most of what goes on :( |
|
Okay, thanks, I just checked how it would look at https://github.com/ddev/ddev |
The Issue
We've been showing long-since expired github sponsors and featured sponsors, and that's not fair to the folks that are currently active.
How This PR Solves The Issue
Manual Testing Instructions
Review the results on the rendered home page
Rendered
It won't show up on https://github.com/ddev/ddev yet of course, but that should be the same.