Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Site] Fix Local Development Setup #977

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

M-DEV-1
Copy link

@M-DEV-1 M-DEV-1 commented Mar 18, 2025

Notes for Reviewers

This PR fixes #976

Previously, users were unable to set up the local development environment to work with the existing Sistent site.

This PR updates the Makefile to include the correct command for setup and removes redundant imports for better maintainability.

Screenshots

Sistent-website-light
Sistent-website-dark

After running make site, users will now see the expected output.

Signed commits

  • Yes, I signed my commits.

M-DEV-1 added 5 commits March 18, 2025 01:54
Signed-off-by: M-DEV-1 <mahadevankizhakkedathu@gmail.com>
Signed-off-by: M-DEV-1 <mahadevankizhakkedathu@gmail.com>
Signed-off-by: M-DEV-1 <mahadevankizhakkedathu@gmail.com>
Signed-off-by: M-DEV-1 <mahadevankizhakkedathu@gmail.com>
Signed-off-by: M-DEV-1 <mahadevankizhakkedathu@gmail.com>
@github-actions github-actions bot added area/docs Improvements or additions to documentation area/site labels Mar 18, 2025
@M-DEV-1
Copy link
Author

M-DEV-1 commented Mar 18, 2025

I'm not sure if the React imports in these files are the correct approach.

I followed the error messages to get the site running, I appreciate any guidance on whether this is the right approach.

@M-DEV-1
Copy link
Author

M-DEV-1 commented Mar 18, 2025

ESLint and Babel are not configured either, which may cause linting and build issues.

Screenshots:

npx gatsby build

image


npx eslint src

image

@amitamrutiya
Copy link
Contributor

amitamrutiya commented Mar 19, 2025

@M-DEV-1 still there is in lot of place error is present
image

disable this type of error else from the lint else fix the errors as we don't want this.

@M-DEV-1
Copy link
Author

M-DEV-1 commented Mar 19, 2025

@M-DEV-1 still there is in lot of place error is present image

disable this type of error from the lint as we don't want this.

I'll set up the ESLint config file for /site and add this as a rule. The ESLint package might be upgraded in the process.

Is this okay?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/docs Improvements or additions to documentation area/site
Development

Successfully merging this pull request may close these issues.

[Site] Error in building site locally
3 participants