Problem/Motivation

Like the title says. move help topics from help_topics to action

Steps to reproduce

N/A

Proposed resolution

Remaining tasks

patch
review
commit

User interface changes

N/A

API changes

N/A

Data model changes

N/A

Release notes snippet

N/A

CommentFileSizeAuthor
#6 3343600-6.patch3.01 KBquietone
#6 interdiff-2-6.txt1.54 KBquietone
#2 3343600-2.patch1.6 KBquietone

Comments

quietone created an issue. See original summary.

quietone’s picture

quietone commented
Status: Active » Needs review
StatusFileSize
new 3343600-2.patch1.6 KB
smustgrave’s picture

smustgrave commented
Status: Needs review » Reviewed & tested by the community
Issue tags: +Needs Review Queue Initiative

Looks good.

danielveza’s picture

danielveza
he/him
Location Brisbane, AU
commented
Status: Reviewed & tested by the community » Needs work

Do we include content fixes as part of this? The actions UI modules help topics reference actions ui being a core module.

I think we should take that out while we are here, but if thats considered out of scope I'm happy for this to stay at RTBC

andypost’s picture

andypost
he/him
Primary language Russian
commented
Related issues: +#3025577: Move help topics to core or the correct modules
quietone’s picture

quietone commented
Status: Needs work » Needs review
StatusFileSize
new interdiff-2-6.txt1.54 KB
new 3343600-6.patch3.01 KB

#4. Yes, I checked the similar issue for Tracker to be sure.

danielveza’s picture

danielveza
he/him
Location Brisbane, AU
commented
Status: Needs review » Reviewed & tested by the community

Nice :). Verified the only change was the suggestion in #4. Back to RTBC

catch’s picture

catch
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Committed 22a39eb and pushed to 10.1.x. Thanks!

  • catch committed 22a39eb4 on 10.1.x 
    Issue #3343600 by quietone, DanielVeza: Move help topics to actions...
catch’s picture

catch
he/him
Primary language English
commented

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.