search

wiktoriavh / learn-web-dev

A Single Page with ressources to learn Web Development from Scratch
learn-web-dev.vercel.app
4 stars 4 forks source link

Basics of the Internet #21

Closed Khushi2402 closed 1 year ago

Khushi2402 commented 2 years ago

Please check now.

vercel[bot] commented 2 years ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
learn-web-dev ✅ Ready (Inspect) Visit Preview Sep 13, 2022 at 7:12PM (UTC)
wiktoriavh commented 2 years ago

You don't need to create a new PR for each change, you can just push to the previous branch and it will update the PR.

The file looks good, I still need time to check it out, though.

wiktoriavh commented 2 years ago

Relevant Issue #11

Khushi2402 commented 2 years ago

@Braweria Sorry for the mistake, I'm still learning. I will keep that in mind in future. Thank you.

wiktoriavh commented 2 years ago

That's alright, the whole point of this Repo is to learn Web Development. :) It also lets you get used to work with PRs and the environment you might find yourself in. @Khushi2402

wiktoriavh commented 2 years ago

I have read most of the articles now, the ones I didn't I haven't commented on yet. I also feel like you didn't put in the effort to read the article, and I noticed you copy-pasted the URL you were given directly after a search from google, since the Highlight was still in the URL. There were also a lot of duplicate articles on the same topic.

While I really appreciate the help in gathering those articles, it is important to know, that you also put effort into reading and checking them. It took me several days now to get through them. Usually a quick look gives a good overview if it's a a good article or not.

Anyway, I have requested changes and will review once again when they are done. @Khushi2402

wiktoriavh commented 1 year ago

If you find more information, that you have read and think is a great resource for beginners, feel free to make a comment in the new PR I created for the issue: #22 There is still stuff missing on other topics mentioned in the issue. I have closed this branch now.