Skip to content
Snippets Groups Projects

Groups - README

Merged Zack Cuddy requested to merge 392015-group-readme-fe into master

What does this MR do and why?

Built off the base MR from @peterhegman: !110943 (closed)
Closes #392015 (closed)

This change hooks up the ability to display a README on the Group Information page via a specially named project gitlab-profile.

This is behind the feature flag (:show_group_readme)

Screenshots or screen recordings

w/ README Project w/o README Project
Group Group_README Group_no_README
Subgroup Subgroup_README Subgroup_no_README

How to set up and validate locally

  1. Enable the Feature Flag Feature.enable(:show_group_readme)
  2. I've had issues lately with FF caching, I'd suggest maybe restarting GDK
  3. Go to a Group or Subgroup
  4. Create a project named exactly gitlab-profile
  5. Add a README.md to the project
  6. Go back to the Group Information page
  7. Ensure README.md is present under the list of Projects/Subgroups

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #392015 (closed)

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Tan Le
  • Tan Le
  • Tan Le
    • Resolved by Tan Le

      Thank you for your work here, @zcuddy 🙇🏻 The change looks good to me and I can confirm that it works as described in my local GDK. I have left some suggestions for your consideration. Please let me know what you think 🙏🏼.

  • Tan Le removed review request for @tle_gitlab

    removed review request for @tle_gitlab

  • mentioned in issue #392016 (closed)

  • Zack Cuddy added 599 commits

    added 599 commits

    Compare with previous version

  • Zack Cuddy requested review from @tle_gitlab

    requested review from @tle_gitlab

  • Zack Cuddy added 1 commit

    added 1 commit

    Compare with previous version

  • Zack Cuddy
  • Fernando Cardenas
  • Fernando Cardenas requested review from @zcuddy

    requested review from @zcuddy

  • Tan Le
  • Tan Le
  • Tan Le removed review request for @tle_gitlab

    removed review request for @tle_gitlab

  • Zack Cuddy added 503 commits

    added 503 commits

    Compare with previous version

  • Peter Hegman mentioned in merge request !110943 (closed)

    mentioned in merge request !110943 (closed)

  • Zack Cuddy requested review from @tle_gitlab and removed review request for @zcuddy

    requested review from @tle_gitlab and removed review request for @zcuddy

  • Tan Le
  • Thanks for 🐻-ing with me, @zcuddy 🙏🏼. The MR is shaping up quite nicely. I have left some suggestions. Please let me know what you think.

  • Fernando Cardenas requested review from @zcuddy

    requested review from @zcuddy

  • Tan Le removed review request for @tle_gitlab

    removed review request for @tle_gitlab

  • Zack Cuddy added 554 commits

    added 554 commits

    Compare with previous version

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading