Skip to content
OverviewSolutions
8
Comments
2

Dario Rodrigues

@UnidadeBrazil160 points

I'm a self-taught web developer from Brazil. I'm here to improve my skills in creating full-fledged websites and I'll be pleased to participate in the awesome front-end mentor community.

I’m currently learning...

React Server Component Redis Database AWS

Latest solutions

View more solutions

Latest comments

  • Dario Rodrigues•160
    @Unidade

    I loved your animations; they were such a great addition to the challenge. Overall, I think it's a pretty solid implementation, and in terms of code, it's well-written and easy to navigate.

    I'm curious about how much time you spent on this challenge

  • Dario Rodrigues•160
    @Unidade

    I looked your javascript code and I found that you a are making more fetch calls than the necessary, I inspected this with the network section on devTools. This is an example that are present in your components:

    {fetchLocation(ip).city}, {fetchLocation(ip).country}

    The response of this fetch call is a data object that have all the properties that you are accessing, you can be more efficient if you store the fetch data in a variable and access them as need. For example, this only make a fetch call.

    const data = fetchLocation(ip)
{data.city}, {data.country}
...

    This is important because you don't want to make unnecessary call to the server, they can bottleneck your app, you are using more of your api call quotes and isn't efficient at all.

    Marked as helpful