Description of the Change

Closes #606

How to test the Change

1. Open an event
2. Add the new "Add to calendar (BUTTONS)" block.
3. Add the new "Add to calendar (DETAILS)" block.
4. Save
5. View in frontend and check that Google & Yahoo buttons work.

Known Issue: ical & outlook have no href attribute.

I guess that WordPress added some escaping to the sources bound to the url attribute of a button.

Because, when I make a debug-change and direct the Block Binding API to output the values into the text attribute, instead of url, one can see that the URLs are properly delivered into the block.

metadata: {
	bindings: {
		text: {
			source: 'gatherpress/add-to-calendar',
			args: {
				service,
			},
		},
	},
},

I think the ical data-uri might just fail against esc_url() or something similar.

I would not recommend trying to fix this or to do any changes to this PR, but instead move #928 forward!

#928 and another, yet unfinished necessary follow-up (to fully align with #831) will introduce new named endoints like https://example.org/event/test-event-954/ical, which will make the escaping pass and the blocks work like expected.

Changelog Entry

Added - New block variation "Add to calendar"

Credits

Props @carstingaxion

Checklist:

• I agree to follow this project's Code of Conduct.
• I have updated the documentation accordingly.
• I have added tests to cover my change.
• All new and existing tests pass.