[ihordubas99]

Ticket

https://community.openproject.org/wp/BNE-148

What are you trying to accomplish?

Wire up the #### hash command that creates a block-sized work package link. The feature itself lives in op-blocknote-extensions (opf/op-blocknote-extensions#246); this is the consumer side of it.

What approach did you choose and why?

The # menu now ships as <OpenProjectHashMenu />, alongside <OpenProjectFormattingToolbar />, so the editor no longer restates the trigger character and the menu's wiring - including what the new #### behaviour needs - stays on the extensions side. Nothing to change here when it grows further.

Needs the op-blocknote-extensions release that exports OpenProjectHashMenu. The dependency bump in frontend/package.json is not in this PR yet - it has to land here before this can be merged, otherwise the frontend build fails on the missing export.

Merge checklist

• Added/updated tests
• Added/updated documentation in Lookbook (patterns, previews, etc)
• Tested major browsers (Chrome, Firefox, Edge, ...)

[github-actions]

Caution

The Enterprise plan field is not set on the work package

Details:

• Work package URL: https://community.openproject.org/wp/BNE-148

Please make sure that:

• The work package Enterprise plan field is set

[judithroth]

Nice improvement with the OpenProjectHashMenu!
We have to wait with merging this until the next release of BNE of course.

I get one failure with the changes of this and opf/op-blocknote-extensions#246 though:

Failures:

  1) BlockNote editor rendering with op-blocknote-extensions orders results of the work package search by updated at DESC
     Failure/Error: expect(editor.element).to have_content("AAA test") # wait for dropdown to open
       expected to find text "AAA test" in ""
     # ./modules/documents/spec/features/block_note_editor_spec.rb:151:in 'block (3 levels) in <top (required)>'
     # ./spec/support/capybara.rb:22:in 'block (2 levels) in <top (required)>'
     # ./spec/support/shared/without_env.rb:52:in 'block (2 levels) in <top (required)>'
     # ./spec/support/shared/with_env.rb:60:in 'block (2 levels) in <top (required)>'
     # ./spec/support/shared/with_direct_uploads.rb:219:in 'block (2 levels) in <top (required)>'
     # ./spec/support/cuprite_setup.rb:164:in 'block (2 levels) in <top (required)>'
     # -e:1:in 'Kernel#require'
     # -e:1:in '<main>'

rspec ./modules/documents/spec/features/block_note_editor_spec.rb:141

[judithroth]

👍

[ihordubas99]

Hi @judithroth, thanks for checking!

I rebased onto dev and the spec passes locally now, together with the new #### tests 👌

[judithroth]

(please do not merge yet, BNE needs to be released first)