another usages added

[ayushkumar1610]

Description

Modification made to usages.rst, a use of browser-history is added. A python scripts to show number of times sites visited using graph.

Fixes #160

Type of change

Please delete options that are not relevant.

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• This change requires a documentation update

Checklist:

• I have read the contribution guidelines and followed it as far as possible.
• I have performed a self-review of my own code (if applicable)
• I have commented my code, particularly in hard-to-understand areas (if applicable)
• I have made corresponding changes to the documentation (if applicable)
• My changes generate no new warnings
• I have enabled the pre-commit hook and it's not detecting any issue.
• Any dependent and pending changes have been merged and published

[Samyak2]

Thanks for the PR! I had a few questions and suggestions.

Could you also link the issue in the description? (Change Fixes # (issue) to Fixes #160, that will link the issue with your PR)

[Samyak2]

Question - what does this do? Why not use por[2] always?

[ayushkumar1610]

Yeah initially I thought of that but, if users have a history of local file file:///D... then the site name will be the empty string.
What do you suggest on this?

[Samyak2]

A better solution would be to use the in-built urllib.parse and get hostname from that. If hostname is None, we can ignore that tuple.

[codecov-commenter]

Codecov Report

Merging #175 (10f924f) into master (8d1135d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #175   +/-   ##
=======================================
  Coverage   92.00%   92.00%           
=======================================
  Files           5        5           
  Lines         475      475           
  Branches       87       87           
=======================================
  Hits          437      437           
  Misses         26       26           
  Partials       12       12           

Flag	Coverage Δ
unittests	92.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8d1135d...10f924f. Read the comment docs.

[Samyak2]

Just noticed a few things. However, I will add the hacktoberfest-accepted tag to approve your PR 🎉. Thanks for contributing.

• Could you move this to the end of the usage section? Something like an "Examples" subsection seems more appropriate for this.
• Would it be possible to take just the top 10 or 20 websites and plot them? Instead of using all websites. This is how the graph looks on my system 😆
  

[Samyak2]

A better solution would be to use the in-built urllib.parse and get hostname from that. If hostname is None, we can ignore that tuple.

[Samyak2]

Should be

Suggested change

	from browser_history.browsers import get_history
	from browser_history import get_history

[Samyak2]

It would be slightly more idiomatic to use:

Suggested change

	for tup in history:
	url = tup[1]
	for _, url in history:

[Samyak2]

The conditions are switched :D

Suggested change

	if site not in frequency:
	frequency[site] += 1
	else:
	frequency[site] = 1
	if site not in frequency:
	frequency[site] = 1
	else:
	frequency[site] += 1

[Samyak2]

@ayushkumar1610 do you want to continue working on this?

[Samyak2]

@ayushkumar1610?