maproulette / maproulette/maproulette-frontend
Rapid URL should be a config parameter (rather than hardcoded)
Nobody has claimed this yet.
- Dominant language
- JavaScript
- Stars
- 167
- Forks
- 39
- Avg merge
- 16h 34m
- Merged PRs (30d)
- 11
Description
As a temporary workaround for https://github.com/facebook/Rapid/issues/1561, I published a hotfix to pin the Rapid version at 2.3. This required changing the MR frontend code, tagging a new version, building a new docker artifact and redeploying the service.
It would have been easier if I had chosen to make the Rapid CDN URL a configurable value when I refactored the editor integration recently (something like REACT_APP_RAPID_EDITOR_CDN_URL in .env). So I'm opening this issue to remind myself to implement that at some point. I'm not sure if it's straightforward to embed values from the config into static assets like rapid-editor.html but I think it's probably possible.
Contributor guide
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 by examining public/static/rapid-editor.html and the frontend's .env configuration, then trace how configuration values are included in built static assets. The work is done when the Rapid CDN URL can be changed through configuration without editing frontend code, tagging a version, or rebuilding the Docker artifact for each URL change.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- build-system, frontend
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100