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

Added support for scraping of NPCs #35

Closed
wants to merge 25 commits into from

Conversation

Jocarnail
Copy link
Contributor

Scrape NPCs taking them from the NPC's table. The implementation of "getCreatureDetails" may be improved:

  • The Id field is taken from the href of the site, that contains the relative url for the creatures.
  • This contains already all the information to fetch the info on the creatures, be them monsters or NPCs.
  • It may be efficient to take as the creature's id the whole url and use it to fetch both monsters and npcs using the same function.

May try to do it in the next days.

dependabot bot and others added 22 commits January 12, 2021 19:42
Bumps [axios](https://github.com/axios/axios) from 0.18.1 to 0.21.1.
- [Release notes](https://github.com/axios/axios/releases)
- [Changelog](https://github.com/axios/axios/blob/v0.21.1/CHANGELOG.md)
- [Commits](axios/axios@v0.18.1...v0.21.1)

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [axios](https://github.com/axios/axios) from 0.18.1 to 0.21.1.
- [Release notes](https://github.com/axios/axios/releases)
- [Changelog](https://github.com/axios/axios/blob/v0.21.1/CHANGELOG.md)
- [Commits](axios/axios@v0.18.1...v0.21.1)

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…into develop

# Conflicts:
#	src/webui/src/pages/Index.vue
…e scraping (use same function) and fix incorrect address for npc in webui.
@Jocarnail
Copy link
Contributor Author

Changed scraping to take the entire relative url of the creatures ad ID: this allows to use the same function to scrape both monsters and npcs. Changes in the webui to be to open the correct page for both when one would like to consult the AoN. Now the address is directed by the id from the domain to the correct page (Eg: "Monsters.aspx" vs "NPCs.aspx")

@maxiride
Copy link
Owner

Closing the PR since the availability of the new python script in #46 should cover this

@maxiride maxiride closed this Aug 16, 2023
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.

2 participants