Thanks to visit codestin.com
Credit goes to github.com

Skip to content

feat: add Zillow module - #396

Open
shredzwho wants to merge 1 commit into
kaifcodec:mainfrom
shredzwho:feature/zillow-module
Open

feat: add Zillow module#396
shredzwho wants to merge 1 commit into
kaifcodec:mainfrom
shredzwho:feature/zillow-module

Conversation

@shredzwho

Copy link
Copy Markdown
Contributor

This PR adds a new shopping module for Zillow.

@kaifcodec

Copy link
Copy Markdown
Owner

@shredzwho We don't need a separate test file for each module, so you can remove it.

One thing we've recently introduced is the extra parameter in Result.taken() to pass any additional data that a site exposes about a user. If this site returns profile metadata publicly, try extracting those details with regex (or another appropriate parser) and pass them through extra.

If the profile data is only available after login or signup, let me know and I will merge it as it is for now.

@kaifcodec kaifcodec added need-extra-changes Some files or parts of the code may require additional edits or updates for it work properly. user module addition PRs or issues related to new user_scan module additions labels Jul 6, 2026
@kaifcodec kaifcodec added the help wanted Extra attention is needed label Jul 28, 2026
@Azaucifer

Copy link
Copy Markdown
Contributor

Hey @shredzwho! 👋

Just checking in on this PR. It's been open for a while and I noticed it has the help wanted label.

I'd be happy to help finish it up by:

  • Removing the test file (as requested)
  • Adding extra data extraction for any public profile metadata

Are you still working on this? If not, I can pick it up and open a new PR with the changes.

Let me know! 🙌

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

help wanted Extra attention is needed need-extra-changes Some files or parts of the code may require additional edits or updates for it work properly. user module addition PRs or issues related to new user_scan module additions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants