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

Empty ministerial links when reshuffle mode is on #9135

Merged
merged 1 commit into from
Jun 11, 2024

Conversation

brucebolt
Copy link
Member

Publishing API doesn't delete links unless you send it an empty array for a specific key.

Therefore updating the presenter to include all the keys that need to be emptied, rather than an empty hash.

Publishing API doesn't delete links unless you send it an empty array
for a specific key.

Therefore updating the presenter to include all the keys that need to be
emptied, rather than an empty hash.
Copy link
Contributor

@ChrisBAshton ChrisBAshton left a comment

Choose a reason for hiding this comment

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

LGTM - thanks Bruce!

@brucebolt brucebolt enabled auto-merge June 11, 2024 13:03
@brucebolt brucebolt merged commit d126cbd into main Jun 11, 2024
20 checks passed
@brucebolt brucebolt deleted the blank-ministers-links branch June 11, 2024 13:06
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.

2 participants