feat: Adding the displayname to the output of occ group:list --info

[ThibautPlg]

Adding the displayname to the output of occ group:list --info command

• Resolves: no bugs related

Summary

When using occ group:list --info, the displayName is not returned. I then need to make a new occ group:info groupId for every group to have a full and usable list.
This PR simply adds the displayName info to the info output.

Before :

# [...]occ group:list --info 
[...]

  - mygroup:
    - backends:
      - Database
    - users:
      - thibautplg

After :

# [...]occ group:list --info 
[...]

  - mygroup:
    - displayName: My Nextcloud group
    - backends:
      - Database
    - users:
      - thibautplg

Checklist

• Code is properly formatted
• Sign-off message is added to all commits
• Tests (unit, integration, api and/or acceptance) are included
• Screenshots before/after for front-end changes
• Documentation (manuals or wiki) has been updated or is not required
• Backports requested where applicable (ex: critical bugfixes)

[Altahrim]

@ThibautPlg Can you rebase and reword your commit using conventional commit please?

[ThibautPlg]

@ThibautPlg Can you rebase and reword your commit using conventional commit please?

Not sure if I merged the correct way. Can the conventional commit rules be specified on the generic contribution checklist?
Regards,

[Altahrim]

@ThibautPlg You changed the title of the pull request but you have to rename the commit itself

As you have a merge commit, you should be able to use the following commands:

git pull
git rebase -i origin/master --sign-off
# Here, drop the merge commit and reword the other commit
git push --force-with-lease

[ThibautPlg]

Reverting because I noticed wrongly merged files in the last merge commit.

[AndyScherzinger]

I fixed the commit message for your convenience @ThibautPlg

so @sorbaugh @icewind1991 @Altahrim can you re-review please and merge if fine with you?

[ThibautPlg]

I fixed the commit message for your convenience @ThibautPlg

so @sorbaugh @icewind1991 @Altahrim can you re-review please and merge if fine with you?

If something was awaiting my input I didn't noticed.
Thanks you

[welcome]

Thanks for your first pull request and welcome to the community! Feel free to keep them coming! If you are looking for issues to tackle then have a look at this selection: https://github.com/nextcloud/server/issues?q=is%3Aopen+is%3Aissue+label%3A%22good+first+issue%22

[AndyScherzinger]

Thanks @ThibautPlg - merged it now. All seemed fine and all reviews are there. Some test won't be run on forks, so let's see if they still turn green on the merged master branch 👍

Sorry for the time it took to get it in but happy that we managed to do so in the end 🚀

[juliushaertl]

CI was failing, pushed adjusted unit tests to #47083

[AndyScherzinger]

Thanks a lot @juliushaertl