Refactor trial concerns in sidebars helper
The following discussion from !151448 (merged) should be addressed:
-
@dstull started a discussion: note (non-blocking):
Placement here feels right. I do not like putting application related code in the
lib/
directory. We have established this pattern for view related classes withapp/controllers/concerns/onboarding/status.rb
as well.While this isn't a pure 'rails' concern, I believe the placement here is ok. Perhaps we should add to our standard docs for https://docs.gitlab.com/ee/development/reusing_abstractions.html to discuss/document the approach.