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

Doc: Block API > Registration: switch markdown to a-tags #50110

Merged
merged 3 commits into from
Apr 27, 2023

Conversation

bph
Copy link
Contributor

@bph bph commented Apr 26, 2023

What?

fixes #50109

Why?

in <div callouts> Links need to be a-tags instead of Markdown
That is only a theory, I am testing. I have not found any information in that regard in the documentation.

How?

switch markdown to a-tags

@bph
Copy link
Contributor Author

bph commented Apr 26, 2023

apologies for the misleading branch name. should be docs/update-block-registration

@bph bph self-assigned this Apr 26, 2023
@Mamaduka Mamaduka added the [Type] Developer Documentation Documentation for developers label Apr 27, 2023
@bph bph requested review from carolinan and gziolo April 27, 2023 07:44
Copy link
Member

@gziolo gziolo left a comment

Choose a reason for hiding this comment

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

Let's give it a try. I wasn't aware that everything inside a div needs to be HTML 😞

@bph
Copy link
Contributor Author

bph commented Apr 27, 2023

Thanks @gziolo, as mentioned, it's just a theory, I am testing :-)

It also seems there is one test failing, but also that it might not be connected to my PR. When you get a minute, could you help me figure out what the next steps are to get this merged?

@gziolo gziolo merged commit e85c90c into trunk Apr 27, 2023
@gziolo gziolo deleted the update/block-registration branch April 27, 2023 09:38
@github-actions github-actions bot added this to the Gutenberg 15.8 milestone Apr 27, 2023
@gziolo
Copy link
Member

gziolo commented Apr 27, 2023

The failing test was unrelated, so I pushed the changes using my extended permissions 😅

@bph
Copy link
Contributor Author

bph commented Apr 27, 2023

Thank you! 🤗

@bph
Copy link
Contributor Author

bph commented Apr 27, 2023

It worked: https://developer.wordpress.org/block-editor/reference-guides/block-api/block-registration/
Screenshot 2023-04-27 at 12 33 35

@gziolo
Copy link
Member

gziolo commented Apr 27, 2023

I see now that links are broken, so we probably need also to use full URLs here or tweak the processing for the handbook page that happens for the developer portal.

@bph
Copy link
Contributor Author

bph commented Apr 27, 2023

I also found the links nor working. I try to figure out what the right link would be in this PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Type] Developer Documentation Documentation for developers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Documentation: Use HTML for links on call out
4 participants