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

fix(weather status): Use tmux-weather's cache var #373

Closed
wants to merge 1 commit into from
Closed

fix(weather status): Use tmux-weather's cache var #373

wants to merge 1 commit into from

Conversation

merlinblack
Copy link

After upgrading I found my weather status was not working.

xamut/tmux-weather looks for '#{weather}' in status-right to replace with the downloaded weather data.

However it can't find it as status-right will only contain '#{E:@catppuccin_status_weather}', and results in only the text '#{weather}' being displayed.

So instead of using the literal '#{weather}' in the status module, use the var that xamut/tmux-wether uses to cache it's results.

Tested on my own systems.

xamut/tmux-weather looks for '#{weather}' in status-right to replace
with the downloaded weather data.
However it can't find it as status-right will only contain
'#{E:@catppuccin_status_weather}'

So instead of using the literal '#{weather}', use the var that
xamut/tmux-wether uses to cache it's results.

Tested on my own systems.
@merlinblack
Copy link
Author

Agrh - this does not quite work. Working on a better fix.

@merlinblack
Copy link
Author

Turns out the easiest way to get this going to is to add '#{E:@catppuccin_status_weather}' to the status-right using the -F flag so that #{weather} ends up in thier, and tmux-weather can then update that to call the actual weather script.

Maybe just needs a doc update.

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.

1 participant