-
Notifications
You must be signed in to change notification settings - Fork 40
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
Service section added #55
base: master
Are you sure you want to change the base?
Conversation
Add what's asked in the PR properly, mention the issue don't just write. @mayankkuamr001 |
Updated @2024-SANDHYA |
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.
Firstly, you were not supposed to copy-paste every content. At least see what the website is about and what are you mentioning there. Like cloud service and all those. The image was for the idea.
Also, I don't want this outside box and spacing b/w the cards are also very less @mayankkuamr001 Another thing issue was never assigned to you without that you cannot create a PR it won't be merged. Do you want me to assign the issue to you now ?
Sorry for inconvenience ... Actually this was my first contribution on github. I am a beginner in open source so I am still on my learning phase.Secondly I just created the template for the service section and I thought the data will be filled by someone else. |
Comment on the issue first @mayankkuamr001 |
Why are you closing the PR, issue is assigned to you @mayankkuamr001 |
I made the changes.Kindly check |
Thank you for your contribution to Dust-Road-Co 🤩🤩
Please mention the following in order to get PR merged 🙌🏻🙌🏻