RFC: Is organization/project approval still relevant?
まだ誰も着手していません。
評価
- 難易度
- 5/5
- 見積もり時間
- 1週間以上
- 初心者へのやさしさ
- 20/100
- issue の種類
- 機能追加
- 明瞭さ
- 説明が足りない
- 活発さ
- 停滞
- 技術スタック
- elixir
- 領域
- authorization, backend
調査の方向性
Issueで説明されている組織とプロジェクトのインデックス/表示アクション、および招待システムから始めます。keep、drop、combined の承認オプションを既存の approved カラムと比較します。承認ポリシーが決定され、実装範囲について合意されれば完了です。
索引モデルが issue の本文から書いたものです。
説明
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.
- 主要言語
- Elixir
- スター
- 234
- フォーク
- 82
- PR マージ指標
- 30日以内にマージされた PR はありません
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
code-corps/code-corps-api のほかの issue
-
難易度 5/5 1週間以上 初心者へのやさしさ 15/100
code-corps/code-corps-api#1385 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 25/100
code-corps/code-corps-api#1384 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
code-corps/code-corps-api#1383 · コメント 2 件 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 35/100
code-corps/code-corps-api#1382 · コメント 2 件 ·
-
Difficulty: Medium Skill: Elixir Time: 4 hours
code-corps/code-corps-api#1372 · コメント 3 件 · 担当者 1 名 ·
code-corps/code-corps-api の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
carverauto/serviceradar#4542 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
phoenixframework/phoenix#6847 ·
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 64/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
beyond-all-reason/teiserver#1584 ·