rokucommunity / rokucommunity/vscode-brightscript-language
Improve Deeplinking
Nobody has claimed this yet.
- Dominant language
- TypeScript
- Stars
- 132
- Forks
- 60
- Avg merge
- 13h 53m
- Merged PRs (30d)
- 20
Description
#4 covered adding deeplink support. With the work that @chrisdp has done to allow dynamic ip specification from a dropdown, it would be beneficial to be able to specify a deeplink without needing to provide the ip. It may make sense to just switch to deeplinkParams key and provide the actual key value pairs since I think you'll pretty much always be doing a deeplink request not input since you can't control when you fire the input call.
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 reviewing the deeplink support from issue #4 and the dynamic IP selection work described here. Determine how deeplink parameters are currently represented and whether the proposed deeplinkParams key can omit the IP; done means a deeplink request can use the selected or supplied parameters without requiring an explicit IP, with behavior covered by the relevant existing tests.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- devtools
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100