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

441/add media via paragraphs #442

Merged
merged 1 commit into from Mar 19, 2024
Merged

441/add media via paragraphs #442

merged 1 commit into from Mar 19, 2024

Conversation

markconroy
Copy link
Member

Closes #441

@markconroy markconroy changed the base branch from 2.x to 3.x February 23, 2024 15:12
Copy link
Member

@ekes ekes left a comment

Choose a reason for hiding this comment

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

Looking at the code for groupmedia how were we using it without this?

Edit: Answer we were calling it in that callback which wasn't getting the Group ID any more. Why was this preferable to enabling groupmedia standard behaviour? Cases where the standard behaviour wasn't calculating the group context correctly would be my first guess.
If we do switch though, we should remove the submit handler.

@finnlewis
Copy link
Member

Sorry, not had a chance yet, but will test this soon.

@ekes ekes changed the base branch from 3.x to 4.x March 5, 2024 15:01
@ekes
Copy link
Member

ekes commented Mar 5, 2024

In one of those how did that work. The call we are making relies on a plugin being returned that <for some time requires that to be checked. So our submit handler needs it. In addition to any groupmedia attempts to attach the correct group.

@ekes
Copy link
Member

ekes commented Mar 5, 2024

And this is bring stuff in from the 2.x / 3.x branches we can't use. Let me rebase it back and make a new (additional PR).

@ekes ekes changed the base branch from 4.x to 3.x March 5, 2024 15:22
@ekes
Copy link
Member

ekes commented Mar 5, 2024

So #443 for 4.x and this one for 3.x

Copy link
Member

@stephen-cox stephen-cox left a comment

Choose a reason for hiding this comment

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

👍

@finnlewis finnlewis merged commit 4ae0ea0 into 3.x Mar 19, 2024
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.

Can't add media via paragraphs fields
4 participants