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

Feat: Indeed Scrapper added #986

Closed
wants to merge 4 commits into from
Closed

Conversation

AKASH722
Copy link
Contributor

Description

The changes made in this pull request include the creation of a Indeed scrapper, implementation of unit tests for the scrapper, and updates to the documentation.

Resolves: [ #881 ]

Checklist

Before submitting this pull request, kindly verify that the ensuing checkpoints have been reached.

  • Have you adhered to the repository's defined coding convention rules?
  • Have you updated the 'documentation.md' file with the method/function documentation?
  • Have you sent a message along with the result or response?
  • Have you used the try-catch technique?
  • Has the method/class been added to the documentation (md file)?

Screenshots

image

image

image

image

image

Additional Notes/Comments

I have created the indeed scrapper please check that out.


I certify that I have carried out the relevant checks and provided the requisite screenshot for validation by submitting this pull request.
I appreciate your contribution.

AKASH722 added 3 commits May 16, 2024 18:38
- Add Indeed Test
- Add in dev-documentation
- Improved Test cases
@AKASH722
Copy link
Contributor Author

It sometimes shows
Failed to fetch URL: https://in.indeed.com/m/jobs?q=java&l=mumbai&start=15
even after giving some jobs as output, it is because it was not able to find jobs on 3rd or 4th page as Indeed only has 5/20 jobs per page.

@nikhil25803
Copy link
Member

Hey @AKASH722, handle the error Failed to fetch URL: https://in.indeed.com/m/jobs?q=java&l=mumbai&start=15 instead of sometimes failing in test file.

@AKASH722
Copy link
Contributor Author

Okay i will work on that and update by Tomorrow.

@AKASH722 AKASH722 marked this pull request as draft May 18, 2024 10:20
@nikhil25803 nikhil25803 closed this Aug 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants