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

Groups #169

Draft
wants to merge 13 commits into
base: main
Choose a base branch
from
Draft

Groups #169

wants to merge 13 commits into from

Conversation

SimonLab
Copy link
Member

@SimonLab SimonLab commented Oct 8, 2022

Create groups:
many_to_many groups - lists
many_to_many groups - people

Create the lists table
Create list controller to add and edit lists
Create item controller to link items to lists
- Add/Remove list linked to an item
- Display lists as badges on main page
Check if person exist before insert
Add schemas and controllers tests
- Add test for item controller managing lists
- Fix #168
Create migration, schema and controller for groups
@SimonLab SimonLab added the in-progress An issue or pull request that is being worked on by the assigned person label Oct 8, 2022
@SimonLab SimonLab self-assigned this Oct 8, 2022
@SimonLab SimonLab added this to To do in Simon's list via automation Oct 8, 2022
@SimonLab SimonLab temporarily deployed to dwylauth October 8, 2022 22:28 Inactive
@codecov
Copy link

codecov bot commented Oct 8, 2022

Codecov Report

Merging #169 (3827047) into main (7d05242) will decrease coverage by 17.52%.
The diff coverage is 50.00%.

@@             Coverage Diff              @@
##              main     #169       +/-   ##
============================================
- Coverage   100.00%   82.47%   -17.53%     
============================================
  Files           13       23       +10     
  Lines          219      331      +112     
============================================
+ Hits           219      273       +54     
- Misses           0       58       +58     
Impacted Files Coverage Δ
lib/app/group_list.ex 0.00% <0.00%> (ø)
lib/app/group_person.ex 0.00% <0.00%> (ø)
lib/app_web/controllers/group_controller.ex 0.00% <0.00%> (ø)
lib/app_web/controllers/group_list_controller.ex 0.00% <0.00%> (ø)
lib/app_web/controllers/group_member_controller.ex 0.00% <0.00%> (ø)
lib/app/group.ex 20.00% <20.00%> (ø)
lib/app/person.ex 77.77% <50.00%> (-22.23%) ⬇️
lib/app/item.ex 100.00% <100.00%> (ø)
lib/app/list.ex 100.00% <100.00%> (ø)
lib/app/list_item.ex 100.00% <100.00%> (ø)
... and 4 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Add functions in Person schema to get the list of groups.
@SimonLab SimonLab temporarily deployed to dwylauth October 9, 2022 14:00 Inactive
@SimonLab SimonLab moved this from To do to In progress in Simon's list Oct 9, 2022
Create controller and templates for group members
@SimonLab SimonLab temporarily deployed to dwylauth October 9, 2022 20:53 Inactive
Add member to group display errors if person not found
@SimonLab SimonLab temporarily deployed to dwylauth October 10, 2022 08:26 Inactive
@nelsonic
Copy link
Member

@SimonLab in light of my comment in dwyl/auth#220 (comment) I made the case that groups belong in the auth application where all personally identifiable info is stored.
I think it makes sense to add groups to the MVP for speed.
But ultimately we should not be duplicating the people table in two places.
Personally identifiable data should exclusively be stored in the Auth App which will be deployed on separate infrastructure.

For the purposes of adding collaboration to the MVP we can add people and groups here,
but this is not where this data will be stored longer term because we don't want to mix the concerns.

@SimonLab SimonLab temporarily deployed to dwylauth October 10, 2022 13:51 Inactive
Create group_list_controller to add lists to groups
Update group name
Delete group
@SimonLab SimonLab temporarily deployed to dwylauth October 10, 2022 15:07 Inactive
@SimonLab SimonLab mentioned this pull request Oct 11, 2022
10 tasks
@SimonLab SimonLab removed the in-progress An issue or pull request that is being worked on by the assigned person label Nov 30, 2022
@SimonLab SimonLab moved this from In progress to To do in Simon's list Nov 30, 2022
@SimonLab SimonLab removed their assignment Nov 30, 2022
@nelsonic nelsonic changed the title Group Groups Dec 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants