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

Communities membership #203

Open
wants to merge 4 commits into
base: communities-communities
Choose a base branch
from
Open

Communities membership #203

wants to merge 4 commits into from

Conversation

openbrian
Copy link
Owner

No description provided.

@openbrian openbrian mentioned this pull request Oct 10, 2022
Draft
@coveralls
Copy link

coveralls commented Oct 18, 2022 •
edited
Loading

Pull Request Test Coverage Report for Build 3270082862

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 208 of 208 (100.0%) changed or added relevant lines in 13 files are covered.
  • 8 unchanged lines in 4 files lost coverage.
  • Overall coverage increased (+0.06%) to 95.57%
Files with Coverage Reduction New Missed Lines %
app/helpers/issues_helper.rb 1 95.24%
app/models/report.rb 1 92.86%
app/models/user.rb 2 98.46%
app/controllers/issues_controller.rb 4 90.38%
Totals Coverage Status
Change from base Build 3270081609: 0.06%
Covered Lines: 6148
Relevant Lines: 6433
💛 - Coveralls

@openbrian openbrian force-pushed the communities-communities branch 5 times, most recently from 8afa8e6 to 93763d0 Compare February 23, 2023 02:11
@openbrian openbrian force-pushed the communities-communities branch 7 times, most recently from a71d408 to 22c4492 Compare March 4, 2023 19:14
@openbrian openbrian force-pushed the communities-membership branch 2 times, most recently from 4002d7b to ba2929a Compare March 24, 2023 11:24
@openbrian openbrian force-pushed the communities-communities branch 6 times, most recently from 5d60113 to 430899f Compare April 19, 2023 03:41
@openbrian openbrian force-pushed the communities-communities branch 3 times, most recently from c7b51f1 to 65caec0 Compare May 31, 2024 01:37
@openbrian openbrian force-pushed the communities-membership branch 2 times, most recently from 7245d7d to fc53e90 Compare July 28, 2024 18:33
@openbrian openbrian force-pushed the communities-membership branch 2 times, most recently from c021efa to 54c8ae8 Compare July 31, 2024 03:06
@openbrian openbrian force-pushed the communities-membership branch 5 times, most recently from 3ddca0f to 2eecc83 Compare July 31, 2024 22:58
@openbrian
Add Community.
e30bbe4 
While a community will eventually have multiple organizers, we need at least one person
to be responsible if an issue is raised.

The community has one owner.  In later PRs, there will be multiple members and multiple organizers.
@openbrian
Create CommunityLinks.
a002e2b 
Links must be https.  Add validate_url ruby gem to validate the urls.

Add not null on link site and url.

Add relevant FK.
@openbrian
Add CommunityMember.
ad362bc 
Allow multiple organizers per community.  There is now just 1 leader of an community.

Allow stepping up if there's no organizer.

Add ability to remove memberships.
@openbrian openbrian force-pushed the communities-communities branch 5 times, most recently from 1f59d7c to 706667e Compare August 9, 2024 15:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@openbrian @coveralls