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

Use unique annotation key for smtp secret checksum (#48) #51

Merged
merged 1 commit into from
Feb 14, 2024
Merged

Use unique annotation key for smtp secret checksum (#48) #51

merged 1 commit into from
Feb 14, 2024

Conversation

lleyton
Copy link
Contributor

@lleyton lleyton commented Mar 6, 2023

Use an unique annotation key for the smtp secret checksum, fixing the conflict mentioned in #48.

@lleyton lleyton changed the title Resolve #48 Use unique annotation key for smtp secret checksum (#48) Mar 6, 2023
@nadiamoe
Copy link
Contributor

Ouch, I totally missed this in my SMTP PR 🤦🏻‍♀️ Thanks for fixing it @lleyton!

@lleyton
Copy link
Contributor Author

lleyton commented Mar 19, 2023

hope it gets merged soon so I don't have to rely on my fork :3

@Azuki-bar
Copy link

Thanks for your fixing! When is this PR likely to be merged?

@gefloh
Copy link

gefloh commented Jun 8, 2023

PR ready, approved, but no one merging this fix leaving the Helm-Chart broken for months.... please merge this.

@wrmilling
Copy link

@timetinytim @renchap Would y'all be able to give this a quick look/merge? Currently the only thing preventing me (and others) from running this chart directly when using Flux CD.

@timetinytim
Copy link
Contributor

@wrmilling No problem, doing so now! Sorry for the overall delay in getting all these PRs merged, I'm finally working through them all :)

@timetinytim timetinytim merged commit ef9e071 into mastodon:main Feb 14, 2024
@wrmilling
Copy link

@timetinytim All good, thanks for jumping on it!

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.

7 participants