Cleanup some unused issuable helper methods
Because issuable_button_hidden?
is called for merge request only, we could simplify the code further and get rid both of issuable_button_hidden?
and issuable_button_visibility
The following discussion from !48244 (merged) should be addressed:
-
@jprovaznik started a discussion: It seems
issuable_button_hidden?
is now used only by this view - I wonder if we could simplify this further by callingmerge_request_button_hidden?
and we could deleteissuable_button_hidden?
andissuable_button_visibility
(but this could be done separately in a follow-up)