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.

Sign up for GitHub

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

Jump to bottom

fix(settings): apply correct conditions to expiration settings #11156

Merged
merged 1 commit into from
Dec 8, 2023
Merged

fix(settings): apply correct conditions to expiration settings #11156

merged 1 commit into from
Dec 8, 2023

Conversation

Antreesy
Copy link
Contributor

@Antreesy Antreesy commented Dec 8, 2023

☑️ Resolves

🖌️ UI Checklist

🖼️ Screenshots / Screencasts

🏚️ Before 🏡 After
image image

🏁 Checklist

  • 🌏 Tested with Chrome, Firefox and Safari or should not be risky to browser differences
  • ⛑️ Tests are included or not possible

@Antreesy Antreesy added this to the 💞 Following Major (29) milestone Dec 8, 2023
@Antreesy Antreesy self-assigned this Dec 8, 2023
@Antreesy
Copy link
Contributor Author

Antreesy commented Dec 8, 2023

/backport to stable28

@Antreesy Antreesy mentioned this pull request Dec 8, 2023
Merged
2 tasks
@Antreesy Antreesy merged commit 734448a into main Dec 8, 2023
36 checks passed
@Antreesy Antreesy deleted the fix/11154/expiration-settings branch December 8, 2023 10:56
@backportbot-nextcloud backportbot-nextcloud bot mentioned this pull request Dec 8, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nickvergessen nickvergessen nickvergessen approved these changes

@DorraJaouad DorraJaouad Awaiting requested review from DorraJaouad

Assignees

@Antreesy Antreesy

Labels
3. to review feature: conversations 👥 feature: frontend 🖌️ "Web UI" client regression
Projects
None yet
Milestone
v19.0.0-beta.1
Development

Successfully merging this pull request may close these issues.

Message expiration, I just cant figure how to use/change it
2 participants
@Antreesy @nickvergessen