-
Notifications
You must be signed in to change notification settings - Fork 15
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: scrape organisation location from ror #1127
Conversation
4db9e85
to
819c161
Compare
819c161
to
ade237d
Compare
e75b370
to
9d0713a
Compare
9d0713a
to
7bc755b
Compare
45575d8
to
f1d7e2b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work!
One suggestion. Maybe we can add slug
of the organisation to link to organisation page directly from error logs (admin section), see 118-backend-logs-views.sql RPC script. This can be done later too.
|
|
I added this feature in the last commit. |
Adds ROR scraper for organisation's city and country
Fixes #806
Changes proposed in this pull request:
How to test:
MAX_REQUESTS_ROR
to.env
docker compose down --volumes && docker compose build --parallel && docker compose up
PR Checklist:
docker-compose.yml