search

layer5io / layer5

Layer5, expect more from your infrastructure
https://layer5.io
Apache License 2.0
861 stars 1.2k forks source link

[UI] Implement the Use cases page #5190

Closed ritiksaxena124 closed 1 month ago

ritiksaxena124 commented 9 months ago

Current Behavior

We don't have a page on the layer5.io site where we list all the use cases of our product

Desired Behavior

Implement the new page for use cases.

Screenshots / Mockups

Figma mockup link

Contributor Resources and Handbook

The layer5.io website uses Gatsby, React, and GitHub Pages. Site content is found under the master branch.

Join the Layer5 Community by submitting your community member form.

Akshun-01 commented 9 months ago

Hey @ritiksaxena124, what all content are you planning to include on this page? Can I get some ideas, to draft a prototype?

balagowda commented 9 months ago

@ritiksaxena124 i would like to work on this.

saikiranpatil commented 9 months ago

i would like to work on this

ritiksaxena124 commented 9 months ago

Thank you @Akshun-01 @balagowda @saikiranpatil for your interest in this issue. We have a rule to assign issues on first come first serve basis but that doesn't stop you from contributing. We have many issues that need attention, you can grab anyone of those. @Akshun-01 we already have a mockup for that page but we need a contributor to implement it so if you are up for implementation then let's get you assigned.

Akshun-01 commented 9 months ago

@ritiksaxena124, yeah sure! Would be my pleasure to help^^

Akshun-01 commented 9 months ago

Hey @ritiksaxena124, where can i access the mockup from?

ritiksaxena124 commented 9 months ago

Any update @Akshun-01?

Akshun-01 commented 9 months ago

Any update @Akshun-01?

I have started working on the page, I'll give an update by tomorrow!

Akshun-01 commented 9 months ago

Hey @ritiksaxena124, here is the update for the use case page

https://github.com/layer5io/layer5/assets/93793691/4502ec4a-07cd-4a22-a485-a9550a1a1e4a

Akshun-01 commented 9 months ago

Hey, @ritiksaxena124 as discussed in yesterday's meeting, I think I should be using an already existing card. Lee gave two for reference:

Which one should we implement ( I find the workshop one to be better as we don't have any external link to follow up to) Just an idea; But I wanted to point out one major thing about the cards used in the workshops page, its "Read More" button is just annoying I think, It messes with the layout of the complete page and we already have individual pages for it so I think the open in window should be re-labeled as the new Read More and we should get rid of the "Read More" which opens it up in the same page.

So what's your say in this and how should we proceed with the issue further?

Rexford74 commented 9 months ago

Hey @ritiksaxena124, please I'm curious. Is this not the same with the solutions page that was designed a while back? Do correct me if I'm wrong...

ritiksaxena124 commented 9 months ago

Hey, @ritiksaxena124 as discussed in yesterday's meeting, I think I should be using an already existing card. Lee gave two for reference:

Which one should we implement ( I find the workshop one to be better as we don't have any external link to follow up to) Just an idea; But I wanted to point out one major thing about the cards used in the workshops page, its "Read More" button is just annoying I think, It messes with the layout of the complete page and we already have individual pages for it so I think the open in window should be re-labeled as the new Read More and we should get rid of the "Read More" which opens it up in the same page.

So what's your say in this and how should we proceed with the issue further?

@Akshun-01, We can use the same card that we have on the workshops page and instead of relabelling the window icon keep it as it is, no need to change that and yes you are right we can hide the "Read More" which opens it up on the same page as the pages that we have are very long and will mess up the page layout (*only for the use cases page).

@Rexford74 this page is different from the Solutions page. How? This page is like a grid view you see on shopping sites where they show grids of cards for products, in our case we will be listing features and the solutions page is a dedicated page that talks about the pain points and their solution.

Akshun-01 commented 9 months ago

Hey @ritiksaxena124, the data in workshop's card is being fed as follows: image

Could you please guide me, how should I implement this?

Akshun-01 commented 8 months ago

Hey @ritiksaxena124, I reused the Workshop Card as was discussed earlier. Actually, in its original implementation, it used GraphQL to retrieve data and its structure was pretty complicated. So I made a local object matching the fields that were required. But the publicURL field is not accepting either URL or local image location, so I am struggling to make that work.

Other than that, it is pretty much complete, but I had to remove most of the part( as it was related to the "learn more" button). UseCases Page Implentation

This is how it is looking (already mentioned about the image bug). I'll make a draft pr for you to see the code too!^^

Akshun-01 commented 8 months ago

@ritiksaxena124 @vishalvivekm Hey, can anyone help with the image bug please!

saurabh100ni commented 8 months ago

@Akshun-01 would you please make a discuss forum post about the help you need and the bug?

Akshun-01 commented 7 months ago

Hey @saurabh100ni, really sorry for the late reply. I created a post regarding the same. Thanks for this!

here is the link to it : https://discuss.layer5.io/t/help-needed-integrating-data-using-graphql-for-use-cases-page/4574

vishalvivekm commented 7 months ago

@Akshun-01 take a look at the PR,https://github.com/layer5io/layer5/pull/5235 I've added commit which dynamically fetches data using query.

iArchitSharma commented 5 months ago

Comment to get assigned

psankhe28 commented 5 months ago

Hi @iArchitSharma , can I work on this issue?

Ashparshp commented 3 months ago

@psankhe28 Any update?

hargunkaur286 commented 2 months ago

@iArchitSharma @Ashparshp Can I work on this issue if no one's working on it?

Ashparshp commented 2 months ago

@hargunkaur286, As we haven't heard anything from @psankhe28, I am unassigning them from the issue and assigning it to you. Please go ahead.

sudhanshutech commented 2 months ago

@hargunkaur286 any luck?

vishalvivekm commented 2 months ago

Folks, I just wanted to point that quite a lot is already done here : https://github.com/layer5io/layer5/issues/5190 I've reopened the mentioned PR, and see what kept it from being merged given that the changes were reviewed a couple of times on the websites call and that we added the support for dynamically querying use-cases: https://github.com/layer5io/layer5/pull/5235/commits/6aa5971dbbf5bec12fddb75adc132c5d3e181b26

hargunkaur286 commented 2 months ago

@hargunkaur286 any luck?

almost done, will be raising pr today