microsoft / microsoft/AzureTRE
firewall docs need updating
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 235
- Forks
- 192
- Avg merge
- 1d 23h
- Merged PRs (30d)
- 13
Description
Describe the bug
As per #4451 the install step is now supported (docs say it isn't)
I feel that the firewall docs can be enhanced to explain how resource.properties, resource.parent.properties (requires a workspace upgrade step to pass a param to a workspace-service) are passed (and that main needs to run first)
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 with the firewall documentation and review issue #4451 for the updated install-step behavior. Trace how resource.properties and resource.parent.properties are passed through the workspace upgrade step to workspace-service, and confirm that main runs first. Done means the docs describe these flows accurately and no longer say the install step is unsupported.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- azure
- Domain
- cloud, documentation
- Issue type
- Documentation
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100