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

Extract title from parsed webpage fetched through URL #10

Merged
merged 10 commits into from
May 1, 2022

Conversation

Ayman161803
Copy link
Contributor

@Ayman161803 Ayman161803 commented Apr 9, 2022

Fixes #6
Here a sample working of URL.hostname functionality.
image

Example results of the logic written

https://github.com/gajagdh
Resulting Name: github
https://www.nitk.ac.in/ahajhsh
Resulting Name: nitk

@shalithasuranga
Copy link
Member

Hey @Ayman161803 Thank you very much for sending this pull request. Seems like the issue that I created is not very clear. I meant "parsing from webpage" from the "from URL" phrase. I've updated the issue: #6

Will you be able to update this pull request accordingly?

Thanks 🎉

@Ayman161803
Copy link
Contributor Author

Ayman161803 commented Apr 16, 2022

Hey @shalithasuranga! I have updated this PR according to changes in issue title.

Methodology Used

Parse the webpage and retrieve HTML title tag using regex manipulations.

Sample inputs and outputs

image

@Ayman161803 Ayman161803 changed the title Extract hostname for title from URL Extract title from parsed webpage fetched through URL Apr 16, 2022
@shalithasuranga
Copy link
Member

Hey @Ayman161803 Thanks. I have updated a few things from the GitHub editor and will double-check those updates locally 🎉

@shalithasuranga shalithasuranga merged commit f23c31a into neutralinojs:master May 1, 2022
shalithasuranga added a commit that referenced this pull request May 2, 2022
* Fix close process on exit

* Removed $ from commands (#8)

* added CHANGELOG.md file (#9)

* Update CHANGELOG.md

* Removed misleading full-stop (#14)

* Update README.md

* Extract title from parsed webpage fetched through URL (#10)

* Removed $ from commands

* Set default title as hostname

* Update readme.md

* Update readme.md

* Update readme.md

* Improved Extraction of title

* parsed webpage and used regex to get title from tiitle tag

* removed unnecessary async

* improved indentation

* Update index.js

Co-authored-by: Shalitha Suranga <shalithasuranga@gmail.com>

* fixed invalid link (#16)

* Fix close process on exit

* Reverted auto formatting

Co-authored-by: Ayman <76606666+Ayman161803@users.noreply.github.com>
Co-authored-by: JASKEERAT SINGH <jaskeerat10000@gmail.com>
Co-authored-by: Shalitha Suranga <shalithasuranga@gmail.com>
Co-authored-by: Wu Tingfeng <wu.tingfeng@u.nus.edu>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Extract title from the URL (by downloading the webpage)
2 participants