-
-
Notifications
You must be signed in to change notification settings - Fork 157
NextJs docs has been added #360
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
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi there! 👋 Thank you for submitting your first pull request to CodeHarborHub. We appreciate your contribution! Our team will review it soon. If you have any questions or need further assistance, feel free to reach out.
Here's the code health analysis summary for commits Analysis Summary
|
Pls do add the GSSOC Label |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Make Sure, You follow our structure
Resources for Guidance
Here are some resources that may be helpful as you contribute to CodeHarborHub:
- Code Harbor Hub Community Features
- Docusaurus Documentation
- React.js Documentation
- Markdown Guide
- MDX Documentation
- Mermaid Documentation
- NextJs Documetation
Resources for DSA Features on Our Site:
@ajay-dhangar Sorry For making it a mistake Now I resolved the changes you have mentioned .You could check it now .Thanks in advance. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have changed the suggestion you have asked for .Thanks in advance.Happy Hacking
@ajay-dhangar Just now saw what the problem is that docker cli is not identifying my image from the format of webp Now I have added it in a local format where it could identify the image and it could run without error I request you to run it again or review the changes I have made.Thank you |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
change for best practice
docs/NextJs/index.md
Outdated
@@ -0,0 +1,66 @@ | |||
--- | |||
id: Next JS |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
id put like next-js
docs/NextJs/index.md
Outdated
@@ -0,0 +1,66 @@ | |||
--- | |||
id: Next JS |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
id put like next-js
docs/NextJs/index.md
Outdated
Edit `app/page.tsx` (or `pages/index.tsx`) file and save it to see the updated result in your browser. | ||
|
||
|
||
<a href="https://nextjs.org/docs/app"><img src="https://github.com/Aksshay88/Macbook/assets/119944779/1c389169-463a-42e1-b4f0-28830ed4b8a8"></a> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bad Practice:
<img src="img.png">
Good Practice:
<img src="img.png" alt="Image name" />
@ajay-dhangar sure will change it and get back to you ASAP Thanks in advance |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Everything is fine. Please focus on XML/JSX syntax best practices.
docs/NextJs/index.md
Outdated
``` | ||
After the prompts, `create-next-app` will create a folder with your project name and install the required dependencies. | ||
|
||
Create a root layout inside `app/layout.tsx` with the required `<html>` and ``<body>`` tags: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bade Practice:
``<body>``
Good Practice:
`<body>`
docs/NextJs/index.md
Outdated
Edit `app/page.tsx` (or `pages/index.tsx`) file and save it to see the updated result in your browser. | ||
|
||
|
||
<a href="https://nextjs.org/docs/app"><img src="emty.png"></a> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bad Practice:
<img src="empty.png">
Good Practice:
<img src="empty.png" />
In JSX, all tags must be self-closing, so it's important to add the closing slash to the <img>
tag.
@ajay-dhangar Completed changing it .I hope your suggestion that you have requested has been rectified and thanks in advance . |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done and fixed issues but content is not good so, next time write more better.
|
@ajay-dhangar Sure Next time will write the content in a detailed manner |
Related Issue
#359
Description
I have added the Next js docs from the scratch where user could see from the first how to download the Nextjs and app routing ,page routing , architecture
Type of PR
Screenshots / Videos (if applicable)
[Attach any relevant screenshots or videos demonstrating the changes]


Checklist
Additional Context
[Include any additional information or context that might be helpful for reviewers.]
Resources for Guidance
Here are some resources that may be helpful as you contribute to CodeHarborHub:
Resources for DSA Features on Our Site: