Qithub-BOT / Qithub-ORG

✅ Qithub サークルのメインリポジトリ。運用/ルール/全リポジトリの Issue など取りまとめています。
https://Qithub.tk/
Creative Commons Attribution Share Alike 4.0 International
10 stars 2 forks source link

OrganizationのメンバーがGithubにサインインしてない人から見えないことの対応策 #93

Closed hidao80 closed 6 years ago

hidao80 commented 6 years ago

報告

表題の件、Githubユーザ以外の人が問い合わせる先のユーザを見つけることができない問題で、 解決策が分かりましたのでご報告します。

自分の案



TL;DR(結論 2018/04/14 現在)

KEINOS commented 6 years ago

おー、PC+別(テスト)アカウントで People にメンバーが表示されるのを確認しました。→ 私も Public に変えてみました。

ただ、モバイル端末だと People タブが出てこないですね。(Android携帯)

対案

https://qithub.tk/ で Qiitadon の OAuth 認証を設け、ログイン後に表示される「Organization参加」ボタンを押すとメンバーに追加される

という機能を設けてどうでしょう?Organization 追加後は https://dev.qithub.tk/ が使えるようになる。

下記 Issue #87 で議論しているのでコメントアウト

ドメインの役割

KEINOS commented 6 years ago

Organization→Peopleページのメンバーリストから、公開範囲をPublicにする。

これを行っておかないと GitHub にログインしても Organization メンバーでないと表示されないようです。

hidao80 commented 6 years ago

https://qithub.tk/ で Qiitadon の OAuth 認証を設け、ログイン後に表示される「Organization参加」ボタンを押すとメンバーに追加される

Qithub-BOT Organizationを製品としてみるとスマートでよいと思います。👍 ですが、BOTがforkされる場合を考慮すると、Github連携をBOT機能の一部にしてしまうのはやりすぎなのではないかと考えます。🤔

もしこの機能を実装した場合、BOTをforkしたとき以下のケースが発生すると想定しています。

  1. fork者がOrganization管理ツールをGithub縛りで登録or無視する判断をしなくてはいけなくなる
  2. ぼっちforkを作る場合、そもそもOrganizationを必要としないため冗長な機能になる

これはBOT本来の目的となる機能ではないのでKISSでないと感じました。

拡張機能としては大変優秀で、チームでBOTを管理したい需要がある場合は非常に有効であるとも考えます。👍

案A

ドメインの役割

👍

KEINOS commented 6 years ago

もしこの機能を実装した場合、BOTをforkしたとき

そうか。ある意味、再帰的な状態になりますね。

案A の

「Organizationに追加」ボタンはBOT(scripts repo)には追加しない

に 👍

BOT(/scripts repo)は https://api.qithub.tk/ に特化させ、ログイン系は https://dev.qithub.tk/ 同様に https://qithub.tk/ のサイトにコーディングするようにしてみます。

hidao80 commented 6 years ago

案A で結論のようなので、問題なければ close 🙇‍♂️💪。

ログイン系を実装した場合、本 issue に close したまま追記していただけると状況が追跡できて良いかと思います。

…追記したら Reopen するんでしたっけ?🤔

KEINOS commented 6 years ago

案A で結論で 👍

https://qithub.tk/ のログイン実装や Organization 参加リクエストも Qithub-ORG に UP しますが、issue 表題の「メンバーがみえない」件に関しては解決済みだと思うので Close し 💪

追記は Close のままでできると思いますが、おそらく別 issue となると思うので、その際にこの issue をメンションするようにしましょう。