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

web_server addon mode init data_path options param #223

Merged
merged 7 commits into from
Mar 10, 2024

Conversation

GeoDerp
Copy link
Contributor

@GeoDerp GeoDerp commented Mar 9, 2024

No description provided.

@GeoDerp
Copy link
Contributor Author

GeoDerp commented Mar 9, 2024

Created from suggestion by @gieljnssns :
#147 (comment)

Working on EMHASS-Add-on PR fr this now.

Copy link

codecov bot commented Mar 9, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.63%. Comparing base (6d01db6) to head (97ae22b).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #223   +/-   ##
=======================================
  Coverage   87.63%   87.63%           
=======================================
  Files           6        6           
  Lines        1707     1707           
=======================================
  Hits         1496     1496           
  Misses        211      211           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@GeoDerp
Copy link
Contributor Author

GeoDerp commented Mar 9, 2024

Will let you know when I am happy with this.

@GeoDerp
Copy link
Contributor Author

GeoDerp commented Mar 9, 2024

@davidusb-geek , there is an issue with the directory structure of the webserver on EMHASS-Add-on (since it uses its own ul). Will try and fix this and add it to this PR.

@GeoDerp
Copy link
Contributor Author

GeoDerp commented Mar 9, 2024

Not my finest work. But it should work. @davidusb-geek

@davidusb-geek
Copy link
Owner

Ok I will merge now and test later today.

@davidusb-geek davidusb-geek merged commit 4c7cbe3 into davidusb-geek:master Mar 10, 2024
16 of 17 checks passed
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.

3 participants