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

issue-5452 markdown shortcut #5495

Merged
merged 11 commits into from
Jun 18, 2024

Conversation

kHAPPY2004
Copy link
Contributor

Fixes #5452
Fixes automarkup shortcut bug for heading and blockquote

Copy link

netlify bot commented Dec 5, 2023

Deploy Preview for volto canceled.

Name Link
🔨 Latest commit 1929826
🔍 Latest deploy log https://app.netlify.com/sites/volto/deploys/6639eda0c81c0100096c6ec0

@stevepiercy
Copy link
Collaborator

@kHAPPY2004 do not open and close multiple pull requests for the same issue. See item 3: https://6.docs.plone.org/contributing/first-time.html#things-not-to-do

@@ -0,0 +1 @@
Fixed markup shortcut for heading and blockquote @kHAPPY2004
Copy link
Collaborator

Choose a reason for hiding this comment

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

What exactly was broken, and how was it fixed? Please include some details, such as what was the original broken behavior and what is the new fixed behavior. These change log entries appear in documentation upon release, so they should be helpful to a developer looking to fix a broken behavior that they observed.

@kHAPPY2004
Copy link
Contributor Author

Hi @stevepiercy, sorry for making multiple pull request. I have improved the changelog, please verify and let me know if i have done it correctly this time.

packages/volto-slate/news/5452.bugfix Outdated Show resolved Hide resolved
Copy link
Collaborator

@stevepiercy stevepiercy left a comment

Choose a reason for hiding this comment

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

I approve the change log. I'll request that a few contributors review your PR. Thank you!

@sneridagh
Copy link
Member

@kHAPPY2004 are you familiar with Cypress tests? Do you imagine to create one for the fix (and for other markdown feature)?

@stevepiercy
Copy link
Collaborator

@kHAPPY2004 more information about Cypress, which Volto uses for Acceptance testing.

@kHAPPY2004
Copy link
Contributor Author

@kHAPPY2004 are you familiar with Cypress tests? Do you imagine to create one for the fix (and for other markdown feature)?

@sneridagh No I am not familiar with cypress yet...

Copy link

netlify bot commented Jan 8, 2024

Deploy Preview for plone-components canceled.

Name Link
🔨 Latest commit f30fa6b
🔍 Latest deploy log https://app.netlify.com/sites/plone-components/deploys/667086fb8178db0008f9e115

@stevepiercy stevepiercy changed the title issue-5452 markdown shourtcut issue-5452 markdown shortcut Mar 5, 2024
@ichim-david
Copy link
Member

@sneridagh If this works and I assume it works since Tiberiu and Razvan accepted the work I think we can live with merging this fix without having a test in order to get this over with.

@sneridagh sneridagh merged commit 826d5fd into plone:main Jun 18, 2024
44 checks passed
sneridagh added a commit that referenced this pull request Jun 19, 2024
* main:
  Remove `api` folder. Add `backend` folder using latest backend best practices (#6110)
  Rename files with wrong extension `js->jsx` when they contain JSX. (#6114)
  Improve shadowing by including the support for `js->jsx` extensions in… (#6113)
  issue-5452 markdown shortcut (#5495)
sneridagh added a commit that referenced this pull request Jun 20, 2024
* main:
  Make dynamic/live teaser without additional requests (#6024)
  Fix issue of duplicated blocks upon pasting an image into the Slate E… (#5818)
  Remove `api` folder. Add `backend` folder using latest backend best practices (#6110)
  Rename files with wrong extension `js->jsx` when they contain JSX. (#6114)
  Improve shadowing by including the support for `js->jsx` extensions in… (#6113)
  issue-5452 markdown shortcut (#5495)
sneridagh added a commit that referenced this pull request Jun 20, 2024
* main:
  Make dynamic/live teaser without additional requests (#6024)
  Fix issue of duplicated blocks upon pasting an image into the Slate E… (#5818)
  Remove `api` folder. Add `backend` folder using latest backend best practices (#6110)
  Rename files with wrong extension `js->jsx` when they contain JSX. (#6114)
  Improve shadowing by including the support for `js->jsx` extensions in… (#6113)
  issue-5452 markdown shortcut (#5495)
sneridagh added a commit that referenced this pull request Jun 26, 2024
* main: (73 commits)
  Release 18.0.0-alpha.36
  Rename missing command
  Image widget PR as breaking (#6125)
  Release @plone/slate 18.0.0-alpha.14
  Release @plone/registry 1.7.0
  Rename Makefile targets (#6104)
  fix: nonContentRoutes diff path (#6102)
  Automatically set the label to `03 type: feature (plip)` for PLIPs (#6122)
  Add ImageWidget with upload/drop/external and inline/widget capabilities (#5607)
  Ensure that sidebar field will not steal focus when metadata is edited (#5983)
  Prevent duplicated UUUIDs in inner blocks when copying container blocks (#6112)
  feat: handle breakList in detached TextBlockEditor (#6106)
  Make dynamic/live teaser without additional requests (#6024)
  Fix issue of duplicated blocks upon pasting an image into the Slate E… (#5818)
  Remove `api` folder. Add `backend` folder using latest backend best practices (#6110)
  Rename files with wrong extension `js->jsx` when they contain JSX. (#6114)
  Improve shadowing by including the support for `js->jsx` extensions in… (#6113)
  issue-5452 markdown shortcut (#5495)
  Release 18.0.0-alpha.35
  Release @plone/types 1.0.0-alpha.16
  ...
ichim-david added a commit that referenced this pull request Jun 29, 2024
Co-authored-by: Steve Piercy <web@stevepiercy.com>
Co-authored-by: Víctor Fernández de Alba <sneridagh@gmail.com>
Co-authored-by: ichim-david <ichim.david@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Broken autoformat / markup shortcuts for headings in Slate text block
6 participants