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.

Sign up for GitHub

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

Jump to bottom

update population #991

Merged
merged 1 commit into from
Apr 5, 2022
Merged

update population #991

merged 1 commit into from
Apr 5, 2022

Conversation

ArtificialOwl
Copy link
Member

  • update population on circle creation
  • display inherited population when running occ circles:manage:list

@ArtificialOwl
Copy link
Member Author

/backport to stable23

@ArtificialOwl
Copy link
Member Author

/backport to stable22

@ArtificialOwl ArtificialOwl force-pushed the fix/noid/update-population-on-new-circle branch from 7ef4832 to 5f80418 Compare April 5, 2022 10:55
@codecov-commenter
Copy link

codecov-commenter commented Apr 5, 2022 •
edited
Loading

Codecov Report

Merging #991 (5f80418) into master (d0fbb21) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

❗ Current head 5f80418 differs from pull request most recent head 1b5b453. Consider uploading reports for the commit 1b5b453 to get more accurate results

@@             Coverage Diff             @@
##             master    #991      +/-   ##
===========================================
- Coverage      0.69%   0.69%   -0.01%     
  Complexity     5716    5716              
===========================================
  Files           289     289              
  Lines         19764   19765       +1     
===========================================
  Hits            138     138              
- Misses        19626   19627       +1
Impacted Files Coverage Δ
lib/Command/CirclesList.php 0.00% <0.00%> (ø)
lib/FederatedItems/CircleCreate.php 0.00% <0.00%> (ø)
lib/Tools/Traits/TConsoleTree.php 0.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d0fbb21...1b5b453. Read the comment docs.

@ArtificialOwl
update population
1b5b453 
Signed-off-by: Maxence Lange <maxence@artificial-owl.com>
@ArtificialOwl ArtificialOwl force-pushed the fix/noid/update-population-on-new-circle branch from 5f80418 to 1b5b453 Compare April 5, 2022 11:43
@ArtificialOwl ArtificialOwl merged commit 1e006fc into master Apr 5, 2022
@delete-merged-branch delete-merged-branch bot deleted the fix/noid/update-population-on-new-circle branch April 5, 2022 14:19
This was referenced Apr 5, 2022
Merged
Merged
@blizzz blizzz mentioned this pull request Apr 7, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mejo- mejo- mejo- approved these changes

@CarlSchwan CarlSchwan Awaiting requested review from CarlSchwan

Assignees
No one assigned
Labels
3. to review enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ArtificialOwl @codecov-commenter @mejo-