Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update the readme #535

Merged
merged 9 commits into from
Aug 18, 2022
Merged

Update the readme #535

merged 9 commits into from
Aug 18, 2022

Conversation

zak39
Copy link
Collaborator

@zak39 zak39 commented Aug 16, 2022

What do you think @dorianne-arawa ?

zak39 added 2 commits August 16, 2022 17:35
Update the readme and add screenshots to prepare the project when I will publish it
Ignore the screenshots folder for the production
@zak39 zak39 added documentation Improvements or additions to documentation draft Don't accept the PR labels Aug 16, 2022
@zak39 zak39 requested a review from dorianne-arawa August 16, 2022 15:37
@LivOriona
Copy link
Collaborator

Here are the changes you should do:

In the presentation section (the first), put the URL link on "our website" instead of paste only the link
image

Requirements: the bullet point should be black and on the same line as the others
image

Limit the app: one -s is missing to WorkspacesManagers (look at the screenshot)
image

Configure groupfolders: add the groupe WorkspacesManagers in addition to GeneralManager
image

Confugure groupfolders : change the line (see screenshot) for "In Settings > Group folders > Group folder admin delegation section, add the groups GeneralManager and WorkspacesManagers."
image

Add General Manager section : instead of this line (see screenshot), put this sentence : "Define which users will be General Managers"
image

In "Development and build", change the line (see screenshot) for "For Nextcloud 21 to 24, build Arawa\Groupfolders"
image

@LivOriona
Copy link
Collaborator

Please review the screenshot as following :

  • reduce the size of the window so that it is less wide
  • cut the screenshot below the Workspaces so that it is not too long
  • change some quotas so that we see something else than "unlimited"

Make a new screenshot, in the Workspace management interface. Make sure that only test users appear (christine dupont, ouma, philippe sautron, rémi, etc)

@zak39
Copy link
Collaborator Author

zak39 commented Aug 17, 2022

It's okay @dorianne-arawa

@zak39 zak39 merged commit ca076b7 into main Aug 18, 2022
@zak39 zak39 deleted the chore/update-the-readme branch August 18, 2022 08:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation draft Don't accept the PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants