Skip to content

[18.0][ADD] webservice_server_env: make server_environment an optional dependency#101

Open
dreispt wants to merge 2 commits intoOCA:18.0from
ursais:18-dr-webservice_server_env
Open

[18.0][ADD] webservice_server_env: make server_environment an optional dependency#101
dreispt wants to merge 2 commits intoOCA:18.0from
ursais:18-dr-webservice_server_env

Conversation

@dreispt
Copy link
Member

@dreispt dreispt commented Jun 15, 2025

Forward port of #42

  • [REF] webservice: server_environment should not be a required dependency
  • [ADD] webservice_server_env: make server_environment an optional dependency

@OCA-git-bot
Copy link
Contributor

Hi @etobella,
some modules you are maintaining are being modified, check this out!

@dreispt dreispt force-pushed the 18-dr-webservice_server_env branch 2 times, most recently from 62ea7a3 to 5ccd5f6 Compare June 15, 2025 14:09
@github-actions
Copy link

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Oct 19, 2025
@dreispt dreispt removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Oct 19, 2025
@dreispt
Copy link
Member Author

dreispt commented Oct 19, 2025

Any reason to hold this back?

@github-actions
Copy link

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Feb 22, 2026
@simahawk
Copy link
Contributor

simahawk commented Mar 4, 2026

Any reason to hold this back?

Just time to review it and test it on real life project 😅
Also this is perfectly in line w/ this roadmap OCA/server-env#258

@@ -0,0 +1 @@
../../../../webservice_server_env No newline at end of file
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

setup folder is a leftover from the porting ;)

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you want to take over and make a superseding PR, since you are working on this right now?

Copy link
Contributor

@simahawk simahawk Mar 4, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not working on it. Here you should just exclude the setup folder from the cherry-pick 😉

Copy link
Contributor

@simahawk simahawk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, mere code review, no test

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stale PR/Issue without recent activity, it'll be soon closed automatically.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants