VigneshDevHub / CampX

"CampX: Discover and share the best campgrounds across Globe, where adventure meets community."
https://campx-1.onrender.com/
MIT License
39 stars 69 forks source link

MAPBOX_TOKEN missing in .env #19

Closed ARRY7686 closed 1 month ago

ARRY7686 commented 1 month ago

Description

In the Installation Guide, the documentation fails to mention the need to create a MAPBOX_TOKEN in the .env file. Without this token, the server will not start, resulting in an error:

Error: Cannot create a client without an access token
    at NodeClient.MapiClient (/path/to/node_modules/@mapbox/mapbox-sdk/lib/classes/mapi-client.js:25:11)

Affected Pages:

Type of Documentation Change

Additional Context

When following the installation steps, users encounter an error due to the missing MAPBOX_TOKEN environment variable. The documentation should explicitly instruct users to include the MAPBOX_TOKEN in the .env file to avoid this issue.

Suggested Improvements

Update the Installation Guide to include instructions for setting up the .env file, specifically adding:

MAPBOX_TOKEN=<your_mapbox_token_here>

This change will ensure that users are aware of the required Mapbox token and avoid runtime errors related to missing environment variables.


ARRY7686 commented 1 month ago

Subject: Request to Contribute to Installation Guide Issue

Hi @Vignesh025 ,

I would like to contribute by addressing it. Could you please assign this issue to me so I can work on it as part of Hacktoberfest?

Looking forward to your response!

Best regards, ARRY7686

Vignesh025 commented 1 month ago

Sure @ARRY7686 , I'm assigning this to you. Proceed!

ARRY7686 commented 1 month ago

hi @Vignesh025 just an update on the progress I have finished the work and the PR will be raised on 1st october as soon as the hacktober fest commences

Vignesh025 commented 1 month ago

Hi @ARRY7686 ,

Thank you for the update! That sounds great—raising the PR on October 1st works perfectly. Looking forward to seeing it once Hacktoberfest starts!

ARRY7686 commented 1 month ago

hi @Vignesh025 i have raised the PR adressing the issue kindly merge it