RFC: Is organization/project approval still relevant?
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 5/5
- Thời gian dự kiến
- Hơn một tuần
- Mức phù hợp với người mới
- 20/100
- Loại issue
- Tính năng
- Độ rõ ràng
- Cần làm rõ
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- elixir
- Lĩnh vực
- authorization, backend
Hướng nghiên cứu
Bắt đầu với các action index/show của organization và project cùng hệ thống lời mời được mô tả trong issue. So sánh các tùy chọn phê duyệt keep, drop và kết hợp với các cột approved hiện có; công việc được coi là hoàn tất khi chính sách phê duyệt đã được quyết định và phạm vi triển khai đã được thống nhất.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
We added an approved column to projects and organizations almost a year ago. I'm not sure if we had the organization invite system in mind when adding it.
The way I see it, the columns could still be relevant.
Keep the columns
- once a user creates an organization and a project, the ember app tells them the two "still need to be approved by an administrator"
- the ember app indicates/shows reduced UI for unapproved projects/organizations
There is then the question of how to most effectively filter out unapproved records
Option A
- the index actions for organizations and projects should only list approved items, if no filters are applied
- the show action and the index actions when filtered to specific ids, should still list all records
- that means unapproved records are unlisted, but can still be reached by direct url, where they are shown in a reduced state
Option B
- index and show should be scoped to current user
- owner can see all approved and their own unapproved records
- regular user can only see approved records
- this means unapproved records are not reachable by regular users
Drop the columns
- Organizations can only be created by admins or via invites - no changes other than normal milestone changes.
- Project creation creates a problem of organization owner being able to create multiple projects once they create the organization. We could fix by
- adding a project limit of 1 to each organization
- we can always manually increase the limit for select organizations, if we want to
A potential problem with dropping the approved columns will be that the user could always do something bad, even a user we trust enough to send an invite to.
Once (hopefully) CodeCorps gets big enough, we will probably have direct org/project creation, not via invites. We'll need an approval system at that point anyway.
We could also do both
We could keep the columns in, but auto-approve an organization created via invite and also maybe auto-approve the first project created by such organization.
- Ngôn ngữ chính
- Elixir
- Star
- 234
- Fork
- 82
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của code-corps/code-corps-api
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 15/100
code-corps/code-corps-api#1385 ·
-
Cannot Create new GitHub App Đang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 25/100
code-corps/code-corps-api#1384 ·
-
Track anon Users that register Đang mở
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 35/100
code-corps/code-corps-api#1383 · 2 bình luận ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 35/100
code-corps/code-corps-api#1382 · 2 bình luận ·
-
Difficulty: Medium Skill: Elixir Time: 4 hours
code-corps/code-corps-api#1372 · 3 bình luận · 1 người được giao ·
Tất cả issue của code-corps/code-corps-api
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
carverauto/serviceradar#4596 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
agentjido/jido_harness#80 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
sevenseacat/cinder#235 ·