Skip to content

⬆️ Replace bobdenotter/weatherwidget by bolt/weatherwidget #3587

Merged
tobias-93 merged 2 commits intomainfrom
change_weatherwidget
Sep 26, 2025
Merged

⬆️ Replace bobdenotter/weatherwidget by bolt/weatherwidget #3587
tobias-93 merged 2 commits intomainfrom
change_weatherwidget

Conversation

@macintoshplus
Copy link
Copy Markdown
Contributor

Use Bolt fork of weatherwidget

@bobvandevijver
Copy link
Copy Markdown
Member

The fork is not yet ready to be used.

@bobvandevijver bobvandevijver marked this pull request as draft July 20, 2025 14:26
@macintoshplus
Copy link
Copy Markdown
Contributor Author

Why?

@bobvandevijver
Copy link
Copy Markdown
Member

See Slack, but also because the namespace must be updated (we do not want to continue to use the bobdenotter namespace). Namespace clashes will become a thing (which is also probably why Bob made used his own namespace), so we need to decide/register which namespaces we use for what.

@macintoshplus
Copy link
Copy Markdown
Contributor Author

What is the new namespace? bolt?

@macintoshplus
Copy link
Copy Markdown
Contributor Author

IMHO, we can use this namespace without risk: Bolt\WeatherWidget

The actual namespace: BobdenOtter\WeatherWidget

The proposed namespace uses the same pattern as Bolt\NewsWidget.

@macintoshplus macintoshplus self-assigned this Jul 23, 2025
@bobvandevijver bobvandevijver marked this pull request as ready for review September 26, 2025 19:11
@tobias-93 tobias-93 merged commit 1163a4f into main Sep 26, 2025
43 of 84 checks passed
@tobias-93 tobias-93 deleted the change_weatherwidget branch September 26, 2025 20:02
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