Remove .avatar-container from group partial

Hey @tedtran2019!

Thank you for your contribution to GitLab. Please refer to the contribution documentation for an overview of the process.

When you're ready for a first review, post @gitlab-bot ready. If you know a relevant reviewer(s) (for example, someone that was involved in a related issue), you can also assign them directly with @gitlab-bot ready @user1 @user2.

At any time, if you need help, feel free to post @gitlab-bot help or initiate a mentor session on Discord. Read more on how to get help.

You can comment @gitlab-bot label <label1> <label2> to add labels to your MR. Please see the list of allowed labels in the label command documentation.

This message was generated automatically. You're welcome to improve it.

added Community contribution workflowin dev labels

assigned to @tedtran2019

This merge request will be counted as part of the running Hackathon!

Check out the Hackathon page for more information!

This message was generated automatically. You're welcome to improve it.

added Hackathon label

added linked-issue label

	1 Warning
	Labels missing: please ask a reviewer or maintainer to add backend to this merge request.

Reviewer roulette

Changes that require review have been detected!

Please refer to the table below for assigning reviewers and maintainers suggested by Danger in the specified category:

Category	Reviewer	Maintainer
backend	`@wandering_person` (UTC+7)	`@engwan` (UTC+8)
frontend	`@thutterer` (UTC+1)	`@jivanvl` (UTC-6)

Please check reviewer's status!

Reviewer is available!
Reviewer is unavailable!

Feel free to override these selections if you think someone else would be better-suited or use the GitLab Review Workload Dashboard to find other available reviewers.

To read more on how to use the reviewer roulette, please take a look at the Engineering workflow and code review guidelines. Please consider assigning a reviewer or maintainer who is a domain expert in the area of the merge request.

Once you've decided who will review this merge request, assign them as a reviewer! Danger does not automatically notify them for you.

If needed, you can retry the danger-review job that generated this comment.

Generated by Danger

I'm not really sure about this. I notice that shared/groups/_list.html.haml calls this haml partial, but oddly enough, deleting this partial really doesn't affect any tests in spec/views/. So I just ran a pipeline to see if removing it causes any tests to fail or not.

I did try searching for instances of render and render partial to search for something that used the group partial, and I could only find list. Then I couldn't find anything that uses list, but I'm probably just really confused.

I feel there's a large probability that I just completely misunderstood the issue haha.

@gitlab-bot label frontend

added frontend label

@gitlab-bot ready

marked this merge request as ready

added workflowready for review label and removed workflowin dev label

Hi Coach @terrichu, this Community contribution is ready for review or needs your coaching.

Do you have capacity and domain expertise to review this? If not, find one or more reviewers and assign to them.
If you've reviewed it, add the workflowin dev label if these changes need more work before the next review.

This message was generated automatically. You're welcome to improve it.

requested review from @terrichu

mentioned in issue gitlab-org/quality/triage-reports#15327 (closed)

changed milestone to %16.8

added component:avatar devopsdata stores grouptenant scale maintenancerefactor pajamasintegrate quick win sectioncore platform severity4 typemaintenance workflowin dev + 1 deleted label and removed workflowready for review label

Reviewer roulette

Changes that require review have been detected!

Please refer to the table below for assigning reviewers and maintainers suggested by Danger in the specified category:

Category	Reviewer	Maintainer
backend	`@rkumar555` (UTC+1)	`@engwan` (UTC+8)
frontend	`@thutterer` (UTC+1)	`@jivanvl` (UTC-6)

Please check reviewer's status!

Reviewer is available!
Reviewer is unavailable!

Feel free to override these selections if you think someone else would be better-suited or use the GitLab Review Workload Dashboard to find other available reviewers.

To read more on how to use the reviewer roulette, please take a look at the Engineering workflow and code review guidelines. Please consider assigning a reviewer or maintainer who is a domain expert in the area of the merge request.

Once you've decided who will review this merge request, assign them as a reviewer! Danger does not automatically notify them for you.

If needed, you can retry the danger-review job that generated this comment.

Generated by Danger

@arfedoro would you take the initial frontend review? I'm not sure what's up with danger (no reviewers were suggested) but I'll go by the MR and Issue labels.

requested review from @arfedoro and removed review request for @terrichu

E2E Test Result Summary

allure-report-publisher generated test report!

e2e-test-on-gdk: test report for e24a2cb8

expand test summary

+------------------------------------------------------------------+
|                          suites summary                          |
+-------------+--------+--------+---------+-------+-------+--------+
|             | passed | failed | skipped | flaky | total | result |
+-------------+--------+--------+---------+-------+-------+--------+
| Plan        | 55     | 0      | 0       | 0     | 55    | ✅     |
| Create      | 54     | 0      | 7       | 0     | 61    | ✅     |
| Verify      | 31     | 0      | 0       | 0     | 31    | ✅     |
| Monitor     | 7      | 0      | 0       | 0     | 7     | ✅     |
| Data Stores | 23     | 0      | 0       | 0     | 23    | ✅     |
| Govern      | 74     | 0      | 0       | 0     | 74    | ✅     |
| Release     | 5      | 0      | 0       | 0     | 5     | ✅     |
| Analytics   | 2      | 0      | 0       | 0     | 2     | ✅     |
| Package     | 4      | 0      | 1       | 0     | 5     | ✅     |
| Manage      | 0      | 0      | 1       | 0     | 1     | ➖     |
+-------------+--------+--------+---------+-------+-------+--------+
| Total       | 255    | 0      | 9       | 0     | 264   | ✅     |
+-------------+--------+--------+---------+-------+-------+--------+

approved this merge request

@arfedoro, thanks for approving this merge request.

This is the first time the merge request has been approved. To ensure we don't only run predictive pipelines, and we don't break master, please start a new pipeline before merging.

Please wait for the pipeline to start before resolving this discussion and set auto-merge for the new pipeline. See merging a merge request for more details.

added pipeline:mr-approved label

requested review from @peterhegman and removed review request for @arfedoro

resolved all threads

added 1 commit

e24a2cb8 - Removed avatar container from _group

Compare with previous version

changed title from Remove {-haml partial app/views/shared/groups/_group.html.haml-} to Remove .avatar-container from group partial

changed the description

approved this merge request

added backend label

resolved all threads

enabled an automatic merge when the pipeline for af92c876 succeeds

merged

@tedtran2019, how was your code review experience with this merge request? Please tell us how we can continue to iterate and improve:

React with a or a on this comment to describe your experience.
Create a new comment starting with @gitlab-bot feedback below, and leave any additional feedback you have for us in the comment.

Interested in learning more tips and tricks to solve your next challenge faster? Subscribe to the GitLab Community Newsletter for contributor-focused content and opportunities to level up.

Thanks for your help!

This message was generated automatically. You're welcome to improve it.