Skip to content
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

267 river module rest api update #339

Open
wants to merge 4 commits into
base: otakaro_digital_twin
Choose a base branch
from

Conversation

caseyli92
Copy link
Member

@caseyli92 caseyli92 commented Dec 12, 2024

DESCRIPTION OF PR:

  • Fix the bug in create datastore layer
  • Fix the bug in create style
  • Update the river module to use the arcgis_rest_api.py script - tested with the Kaiapoi AOI (selected polygon)

Closes:

Developer Checklist

  • Make code change
  • Update tests
    • Update / create new tests
    • Ensure these tests have the expected behaviour
    • Test locally and ensure tests are passing
  • Update documentation
    • Readme
    • Docstrings
    • Comments
    • Wiki

Reviewer Checklist

  • Check new code for code smells
  • Check new tests
    • Ensure adequate coverage
    • Check for code smells within tests
  • Check if documentation needs updating
    • Readme
    • Docstrings
    • Comments
    • Wiki

@caseyli92 caseyli92 linked an issue Dec 12, 2024 that may be closed by this pull request
@caseyli92 caseyli92 self-assigned this Dec 12, 2024
@caseyli92 caseyli92 requested a review from LukeParky December 12, 2024 02:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update the river module to utilize the arcgis_rest_api.py script
1 participant