Skip to content

Conversation

@miaulalala
Copy link
Contributor

@miaulalala miaulalala commented Dec 15, 2023

Summary

The OOO status set the predefined vacation status which is awkward when you're out sick 🤧

Before After
image image

It also solves the issue of an automation not running at all when the last_run timestamp is in the future.

Checklist

@miaulalala
Copy link
Contributor Author

/backport to stable28

@miaulalala miaulalala requested review from a team, Altahrim, ArtificialOwl, ChristophWurst, GretaD and icewind1991 and removed request for a team December 15, 2023 10:19
Copy link
Contributor

@GretaD GretaD left a comment

Choose a reason for hiding this comment

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

looks better

@miaulalala miaulalala changed the title fix(ooo): add new ooo status with new emoji fix(ooo): add new ooo status with new emoji and fix last_run timestamp issue Dec 15, 2023
Copy link
Member

@nickvergessen nickvergessen left a comment

Choose a reason for hiding this comment

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

All good apart from the version in the docs

@miaulalala miaulalala changed the title fix(ooo): add new ooo status with new emoji and fix last_run timestamp issue fix(ooo): add new ooo status with new emoji Dec 18, 2023
@miaulalala
Copy link
Contributor Author

All good apart from the version in the docs

This is ok to go in to 28.0.1

Copy link
Member

@nickvergessen nickvergessen left a comment

Choose a reason for hiding this comment

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

LGTM didn't test

@miaulalala
Copy link
Contributor Author

@blizzz approval for 28.0.1

@blizzz blizzz mentioned this pull request Mar 5, 2024
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.

[Bug]: OOO always saves the 🌴 emoji

5 participants