Validate that block config includes at least one newsletter

It turned out that I had to select at least one newsletter under 'Visible Newsletters' on the block config page. Now everything seems to be functioning as expected. I hope this helps someone who comes across such an issue.

Perhaps it's better if the block wouldn't save without selecting a newsletter or at least a warning message is shown on the block config page.

Adjusted the issue text to show what really needs to be done.

Hi, I followed the Steps to reproduce the issue I found that for anonymous it also shows that You are already subscribed to resolve this issue I have Added a message on the block admin page to select a newsletter and ' Visible Newsletters ' a required field.
please review the patch.

Thanks @himanshu_jhaloya! As I commented in the other issue, manually attaching patches is being phased out, and you should create a patch and Merge Request with the Gitlab integration.

DrupalCI and all patch testing will be turned off on July 1, 2024 →

Created the Mr for the same please Review

Nice, thanks @himanshu_jhaloya I really appreciate it.

Thank you @himanshu_jhaloya but this patch while addressed the required select on the block config page, it's causing the below error on trying to access any page that has the block:

Error: Call to undefined method
Drupal\Core\Field\Plugin\Field\FieldWidget\OptionsButtonsWidget::setAvailableNewsletterIds() in Drupal\simplenews\Form\SubscriptionsBlockForm->form() (line 131 of ..... \web\modules\contrib\simplenews\src\Form\SubscriptionsBlockForm.php).

The requirement is that you must set either newsletters or default_newsletters. Either one can be blank, but not both.

AdamPS → changed the visibility of the branch 3421481-you-are-already to hidden.

AdamPS → changed the visibility of the branch newsletter to hidden.

Automatically closed - issue fixed for 2 weeks with no activity.