readthedocs / readthedocs/ext-theme
Integrations: use correct URL in webhook intergration detail page
Nobody has claimed this yet.
- Dominant language
- HTML
- Stars
- 12
- Forks
- 6
- Avg merge
- 13h 25m
- Merged PRs (30d)
- 1
Description
This is likely using PRODUCTION_DOMAIN I'm assuming, and so the URL shown on the page references the beta instance URL instead:
We probably don't want anyone every using that URL (though it will work just the same). This will be hard to support in the future, if we ever get rid of the beta instance.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start at the webhook integration detail page and inspect where PRODUCTION_DOMAIN is used to build the displayed URL. Confirm the page uses the production instance URL rather than the beta instance URL, then verify the corrected URL in the integration details view.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- html
- Domain
- frontend
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100