add 4 extra featured projects to explore page (closes #971)

[SouravInsights]

Description:

Added accountIds of 4 extra projects (nice-node, startkenet.js, snapshot & svelte-stepper) to the mainnet array of FEATURED_PROJECT_ACCOUNT_IDS.

Screenshots:

Note:

I'm not sure about the criteria to select featured projects, but as far as I understand, the consensus is to strike a balance between showcasing well-known projects and giving visibility to lesser-known projects that might deserve attention. Choosing popular projects potentially increase engagement within the platform while promoting newer or less recognized projects, foster diversity & encourage community growth.

But currently, I have added Nice Node, Snapshot & Starknet.js and all 3 projects are well known so let me know if I need to include any other projects to the list. From a quick glance, I think ens-normalize.js is a cool one! Haven't explored all other projects as of now... Let me know your thoughts! @brandonhaslegs @efstajas

closes #971

[netlify]

✅ Deploy Preview for drips-app-v2 ready!

Name	Link
🔨 Latest commit	83d1a35
🔍 Latest deploy log	https://app.netlify.com/sites/drips-app-v2/deploys/65e8387190d9360008aab683
😎 Deploy Preview	https://deploy-preview-993--drips-app-v2.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[brandonhaslegs]

Hey thanks! Someone will take a look today. Can you mention which issue this closes so that it'll close the issue when the PR is merged?

[SouravInsights]

Hey thanks! Someone will take a look today. Can you mention which issue this closes so that it'll close the issue when the PR is merged?

Sure! This is the issue: #971 closes #971

[brandonhaslegs]

Sorry btw I meant something like this:

closes #971

[SouravInsights]

Sorry btw I meant something like this:

closes #971

oh right right.. sorry!!

[efstajas]

thanks a lot! nice selection already.

Let's maybe replace my project svelte-stepper with https://drips.network/app/projects/github/graphprotocol/graph-node, otherwise LGTM.

[evvvritt]

thanks! can you please put closes #971 in the title of this PR so when we squash and merge it will auto-close the issue?

[brandonhaslegs]

thanks! can you please put closes #971 in the title of this PR so when we squash and merge it will auto-close the issue?

I think it still works as a comment right? We talked about that further up there 👆

[jo-elimu]

https://github.com/drips-network/app/actions/runs/8002781144/job/21856728766?pr=993

[evvvritt]

No I think it needs to be in the title to auto-close (since title is used as the commit message which triggers the auto-close) @brandonhaslegs

[evvvritt]

Code correct but is stacking 8 projects on mobile too tall? @brandonhaslegs

[SouravInsights]

thanks a lot! nice selection already.

Let's maybe replace my project svelte-stepper with https://drips.network/app/projects/github/graphprotocol/graph-node, otherwise LGTM.

No no, that's not what I meant! I mean we should definitely give visibility to lesser-known projects by individuals too. If all the featured projects are well known & popular then it would be not as diverse & maybe a bit misaligned. Bdw, I just checked out svelte-stepper & I think it's a cool project that deserve attention!

[SouravInsights]

Code correct but is stacking 8 projects on mobile too tall? @brandonhaslegs

I can think of 2 ideas..

• Pagination: We can split the 8 cards into smaller subsets, likes 4 cards per page and have controls to navigate between pages.

• Carousel: We can also have an horizontal carousel that allows users to swipe left or right to view cards. Although it's still a long list to scroll through but I think users are accustomed to swiping left or right to navigate through content on mobile devices, which makes horizontal scrolling feel natural and intuitive.

[brandonhaslegs]

Good point. I think a horizontal overflow would be fine in mobile but need to design something. You can also suggest something directly in code if you'd like.

[efstajas]

No I think it needs to be in the title to auto-close (since title is used as the commit message which triggers the auto-close) @brandonhaslegs

it doesn't actually, just needs to be in the initial PR description. just added it there :)

No no, that's not what I meant! I mean we should definitely give visibility to lesser-known projects by individuals too. If all the featured projects are well known & popular then it would be not as diverse & maybe a bit misaligned. Bdw, I just checked out svelte-stepper & I think it's a cool project that deserve attention!

I got that, all good! I just think graph node should be featured for sure generally (because it's a big name), and am not super sure that featuring a project by one of the core maintainers of Drips is necessarily the best look, though I really appreciate that you like it 🙌

Code correct but is stacking 8 projects on mobile too tall?

... We could also just hide everything after the first 4 projects on mobile with a media query. Maybe a bit strange to have different content depending on screen size but I think in this instance it could be OK.

[SouravInsights]

I've tried something simple to showcase all the 8 featured projects for smaller screen sizes. Do let me know your feedback! @brandonhaslegs @efstajas

Recording: https://www.loom.com/share/f6be05a3fa7b40df964cdce7b3feb25f?sid=6f64f1bb-2afe-4e77-9c20-512620d4e9fd

[SouravInsights]

Also, replaced svelte-stepper with graph-node

[jo-elimu]

There is a problem with "Build GQL types":

[STARTED] Load GraphQL schemas
[FAILED] 
[FAILED]         Failed to load schema from https://drips-api-mainnet-s1.up.railway.app/:
[FAILED] 
[FAILED]         Context creation failed: Unauthorized
[FAILED]         GraphQLError: Context creation failed: Unauthorized

[efstajas]

@jo-elimu The reason this happens is because PRs from forked repos cannot access the target repo's secrets. In this case, it's failing, because one of those secrets is the GQL access token.

I'm not super sure how to fix this atm. Looking into it.

[efstajas]

Alright, the test thing has been fixed, tests should pass now 👌

[SouravInsights]

Have you guys tested this? Let me know if there's any issues with merging this @efstajas @evvvritt

[brandonhaslegs]

Hey, so the overflow for this should be horizontal not vertical.

[SouravInsights]

Hey, so the overflow for this should be horizontal not vertical.

Yeah making it vertically scrollable was intentional.. But if you think horizontal is better then I'll make the necessary changes..

[brandonhaslegs]

Yeah I think horizontal scroll is normal behavior, like on Instagram when you post multiple photos

[SouravInsights]

Yeah I think horizontal scroll is normal behavior, like on Instagram when you post multiple photos

cool, just pushed the changes..

[efstajas]

LGTM. 👍

Ideally the scrolling would snap to project cards, but we can follow up with this.