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

Release/now 2.4 #34

Open
wants to merge 27 commits into
base: v2.4.3
Choose a base branch
from
Open

Release/now 2.4 #34

wants to merge 27 commits into from

Conversation

yudetamago
Copy link
Member

@yudetamago yudetamago commented Aug 3, 2018

(確認用のPRなのでマージする必要はないです)

v.2.4.3との差分確認用

Omniauth関連

  • release/now-2.2 で使っている omniauth-githubomniauth-facebook のバージョンがmastodon v2.4.3 だとそのままは使えないので、 webpush, jwt, faraday をダウングレードして対応 ( 881365f#diff-e79a60dc6b85309ae70a6ea8261eaf95 )
  • mastodon標準のomniauth実装 ( Omniauthable ) は User ではなく Identity モデルに情報を持たせるようになっていて release/now-2.2 の実装と互換性がないので使用せず
  • omniauth経由で User を新規作成するとき、 User の保存前に user#create_accountAccount を作ろうとするとエラーになるので、 user#build_account を使って User の保存と同時に保存されるように変更 ( 881365f#diff-4676c008b11a5480d73d4a6de01e45b9R365 )

DBのマイグレーション関連

  • mastodon v2.4.3 と release/now-2.2 でマイグレーション済みのファイルが違うので、ファイルを1つずつ指定してマイグレーションする必要あり。例えば以下を実行する。
#!/bin/sh

CMD=`bundle exec rake db:migrate:status | grep down | awk '{print $2}'`
for version in ${CMD}
do
  bundle exec rake db:migrate:up VERSION=${version}
done
  • 途中のrevertするだけのmigration ( def604c ) はエラーになるだけで不必要なのでコメントアウト

ANNotunzdY and others added 26 commits August 3, 2018 14:25
Add daemon.js

fix facebook login button style

github認証用gem追加

omniauthから取得したhashmapの取り扱いを修正

github認証処理を追加

* provider名に応じてusernameを自動生成
* userオブジェクトを取得できれば成功、ログイン
* set_flash_messageも実行し、成功した旨のメッセージを設定する。ただしlayoutに設定されていないため表示されない

「ソーシャルログイン」表示にGitHub用ボタンを追加

メールアドレスの重複等によるGithubログイン失敗時のエラー表示を実装

GitHub認証と同様のエラー表示処理をFacebook認証に追加

SNS連携時、アイコンを自動取得する

add omniauth-github to Gemfile
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants