Skip to content

SMA Sunny Island Bat control#3325

Closed
seaspotter wants to merge 3 commits intoopenWB:masterfrom
seaspotter:feature_sma_bat
Closed

SMA Sunny Island Bat control#3325
seaspotter wants to merge 3 commits intoopenWB:masterfrom
seaspotter:feature_sma_bat

Conversation

@seaspotter
Copy link
Copy Markdown
Contributor

https://forum.openwb.de/viewtopic.php?p=142615#p142615

Ungetestet, alternativ merge in feature_bat_control?

@LKuemmel LKuemmel changed the base branch from master to feature_bat_control April 28, 2026 13:03
@LKuemmel LKuemmel changed the base branch from feature_bat_control to master April 28, 2026 13:03
@LKuemmel
Copy link
Copy Markdown
Contributor

LKuemmel commented Apr 28, 2026

Ich habe mal die Funktion ausprobiert, das Ziel eines PRs zu ändern - keine gute Idee.
Ich habe das Ziel deines PRs auf feature_bat_control umgestellt, leider habe ich damit deinen PR zerschossen. Das Zurückstellen auf master hat es nicht besser gemacht. Gerne kann ich es in feature_bat_control mergen. Kannst Du bitte einen neuen PR erstellen?
EDIT: Ich habe eigenen Branch erstellt: https://github.com/openWB/core/tree/feature_bat_control_sunny_island Dann ist es sauber getrennt.

@LKuemmel LKuemmel closed this Apr 28, 2026
@seaspotter
Copy link
Copy Markdown
Contributor Author

Ich habe mal die Funktion ausprobiert, das Ziel eines PRs zu ändern - keine gute Idee. Ich habe das Ziel deines PRs auf feature_bat_control umgestellt, leider habe ich damit deinen PR zerschossen. Das Zurückstellen auf master hat es nicht besser gemacht. Gerne kann ich es in feature_bat_control mergen. Kannst Du bitte einen neuen PR erstellen?

Ich hatte das auch schon probiert indem ich ihn auf feature_bat_control mergen wollte, dann hat er aber auch alle commits mitgezogen, die im feature noch fehlen, weil mein branch schon weiter ist als das feature branch. Kannst du das feature branch vllt auf den aktuellen Stand hochziehen, dann stell ich den PR gern nochmal aufs feature branch.

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.

2 participants