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

OSS Gate Meetup: Tokyo: 2017-07-13 #478

Closed
koic opened this Issue Jul 13, 2017 · 19 comments

Comments

Projects
None yet
8 participants
@koic
Member

koic commented Jul 13, 2017

東京ミートアップ2017-07-13

このissueは何のために使うの?

ミートアップの始めと終わりに誰が来て、何をして、どうだったかを共有するために使います!

いつ共有をするの?

当日の始めと終わりに、共有する時間をそれぞれ設けます!

どうやって共有するの?

以下のことを両方やってください!

  • 会場で口頭で喋る
  • このissueにコメントで書く

ミートアップの始めに喋る/書くとよさそうなこと

以下に示す内容のいずれかで大丈夫です!

  • 呼んでほしい名前
  • 普段やっていること
  • 普段利用/開発しているもの
  • 当日やること
  • 「よろしくお願いします!」

ミートアップの終わりに喋る/書くとよさそうなこと

以下に示す内容のいずれかで大丈夫です!

  • やったことの進捗
  • ミートアップの感想
  • 「お疲れ様でした!」
@koic

This comment has been minimized.

Show comment
Hide comment
@koic

koic Jul 13, 2017

Member

@koic Ruby 関係の OSS へのパッチを書く予定です。

Member

koic commented Jul 13, 2017

@koic Ruby 関係の OSS へのパッチを書く予定です。

@yasulab

This comment has been minimized.

Show comment
Hide comment
@yasulab

yasulab Jul 13, 2017

Member

今日やること

Member

yasulab commented Jul 13, 2017

今日やること

@highwide

This comment has been minimized.

Show comment
Hide comment
@highwide

highwide Jul 13, 2017

今日やること

  • 初参加なので、まずは感覚をつかみます...!

highwide commented Jul 13, 2017

今日やること

  • 初参加なので、まずは感覚をつかみます...!
@selmertsx

This comment has been minimized.

Show comment
Hide comment
@selmertsx

selmertsx Jul 13, 2017

Contributor

今日やること

  • railsのcustom_validatorにて、ドキュメントにある方法ではstrictを使えないので、それに関して何かする。

課題だと思ったこと

http://guides.rubyonrails.org/active_record_validations.html#strict-validations

custom validatorでは下記のように記載しても、strict validationsが効いてくれません。

class EmailValidator < ActiveModel::EachValidator  
  def validate_each(record, attribute, value)    
    unless value =~ /\A([^@\s]+)@((?:[-a-z0-9]+\.)+[a-z]{2,})\z/i      
      record.errors[attribute] << (options[:message] || "is not an email")    
    end  
  end
end 

class Person < ApplicationRecord  
  validates :email, presence: true, email: true, strict: true
end

これの解決策としては、カスタムバリデータを下記のようにすることです。

class EmailValidator < ActiveModel::EachValidator  
  def validate_each(record, attribute, value)    
    unless value =~ /\A([^@\s]+)@((?:[-a-z0-9]+\.)+[a-z]{2,})\z/i      
      record.errors.add(attribute, (options[:message] || "is not an email"), strict: true )
    end  
  end
end 

僕はドキュメント見ただけでは、strictが使えないことに気づけませんでした。
これに今日の工数の2時間を費やしました!
次、同じことをやろうとした人がハマらないような何かを出来れば良いなと思ってます。

対応案その1

サンプルコードを修正する。

class EmailValidator < ActiveModel::EachValidator
  VALID_OPTIONS = [:strict]

  def validate_each(record, attribute, value)
    unless value =~ /\A([^@\s]+)@((?:[-a-z0-9]+\.)+[a-z]{2,})\z/i      
      errors_options = options.slice(*VALID_OPTIONS)
      record.errors.add(attribute, (options[:message] || "is not an email"), errors_options )
    end  
  end
end 

背景知らない人からしたら冗長ですよね...

対応案その2

errors.add にて、optionsを入れられるよーみたいな話を記載する。

http://guides.rubyonrails.org/active_record_validations.html#errors-add

見るもの

http://edgeguides.rubyonrails.org/contributing_to_ruby_on_rails.html#contributing-to-the-rails-documentation

と過去のrails guidesに関するPRを見る。
[ci skip も入れる。

その他

一応こっちのドキュメントには、書いてある。
http://api.rubyonrails.org/classes/ActiveModel/Errors.html#method-i-add

Contributor

selmertsx commented Jul 13, 2017

今日やること

  • railsのcustom_validatorにて、ドキュメントにある方法ではstrictを使えないので、それに関して何かする。

課題だと思ったこと

http://guides.rubyonrails.org/active_record_validations.html#strict-validations

custom validatorでは下記のように記載しても、strict validationsが効いてくれません。

class EmailValidator < ActiveModel::EachValidator  
  def validate_each(record, attribute, value)    
    unless value =~ /\A([^@\s]+)@((?:[-a-z0-9]+\.)+[a-z]{2,})\z/i      
      record.errors[attribute] << (options[:message] || "is not an email")    
    end  
  end
end 

class Person < ApplicationRecord  
  validates :email, presence: true, email: true, strict: true
end

これの解決策としては、カスタムバリデータを下記のようにすることです。

class EmailValidator < ActiveModel::EachValidator  
  def validate_each(record, attribute, value)    
    unless value =~ /\A([^@\s]+)@((?:[-a-z0-9]+\.)+[a-z]{2,})\z/i      
      record.errors.add(attribute, (options[:message] || "is not an email"), strict: true )
    end  
  end
end 

僕はドキュメント見ただけでは、strictが使えないことに気づけませんでした。
これに今日の工数の2時間を費やしました!
次、同じことをやろうとした人がハマらないような何かを出来れば良いなと思ってます。

対応案その1

サンプルコードを修正する。

class EmailValidator < ActiveModel::EachValidator
  VALID_OPTIONS = [:strict]

  def validate_each(record, attribute, value)
    unless value =~ /\A([^@\s]+)@((?:[-a-z0-9]+\.)+[a-z]{2,})\z/i      
      errors_options = options.slice(*VALID_OPTIONS)
      record.errors.add(attribute, (options[:message] || "is not an email"), errors_options )
    end  
  end
end 

背景知らない人からしたら冗長ですよね...

対応案その2

errors.add にて、optionsを入れられるよーみたいな話を記載する。

http://guides.rubyonrails.org/active_record_validations.html#errors-add

見るもの

http://edgeguides.rubyonrails.org/contributing_to_ruby_on_rails.html#contributing-to-the-rails-documentation

と過去のrails guidesに関するPRを見る。
[ci skip も入れる。

その他

一応こっちのドキュメントには、書いてある。
http://api.rubyonrails.org/classes/ActiveModel/Errors.html#method-i-add

@swamp09

This comment has been minimized.

Show comment
Hide comment
@swamp09

swamp09 Jul 13, 2017

Member

今日やりたいこと

https://github.com/sunspot/sunspot
へのワーニングを抑えるPRを出し終える。

Member

swamp09 commented Jul 13, 2017

今日やりたいこと

https://github.com/sunspot/sunspot
へのワーニングを抑えるPRを出し終える。

@satoryu

This comment has been minimized.

Show comment
Hide comment
@satoryu

satoryu Jul 13, 2017

Member

今日やること

Member

satoryu commented Jul 13, 2017

今日やること

@jjjjpppp

This comment has been minimized.

Show comment
Hide comment
@jjjjpppp

jjjjpppp Jul 13, 2017

今日やること

jjjjpppp commented Jul 13, 2017

今日やること

@yasulab

This comment has been minimized.

Show comment
Hide comment
@yasulab

yasulab Jul 13, 2017

Member

PR出しました 😸

Add link to YouTube playlist
oss-gate/oss-gate.github.io#46

Member

yasulab commented Jul 13, 2017

PR出しました 😸

Add link to YouTube playlist
oss-gate/oss-gate.github.io#46

@satoryu

This comment has been minimized.

Show comment
Hide comment
@satoryu

satoryu Jul 13, 2017

Member

おもむろに @kou さんからネタをもらって、バグチケットを作った。 https://bugs.ruby-lang.org/issues/13744

Member

satoryu commented Jul 13, 2017

おもむろに @kou さんからネタをもらって、バグチケットを作った。 https://bugs.ruby-lang.org/issues/13744

@yasulab

This comment has been minimized.

Show comment
Hide comment
@yasulab

yasulab Jul 13, 2017

Member

今日ご参加させている皆さん: @kou @koic @highwide @selmertsx @swamp09 @satoryu @jjjjpppp

先ほど撮影したミートアップの雰囲気を伝える動画を限定公開してみました 📹 もし問題なければ一般公開しようと思いますが、何かあればご連絡いただけると嬉しいです 🙏

OSS Gate Meetup の雰囲気
https://youtu.be/vjT2BbHzlUM

Member

yasulab commented Jul 13, 2017

今日ご参加させている皆さん: @kou @koic @highwide @selmertsx @swamp09 @satoryu @jjjjpppp

先ほど撮影したミートアップの雰囲気を伝える動画を限定公開してみました 📹 もし問題なければ一般公開しようと思いますが、何かあればご連絡いただけると嬉しいです 🙏

OSS Gate Meetup の雰囲気
https://youtu.be/vjT2BbHzlUM

@yasulab

This comment has been minimized.

Show comment
Hide comment
@yasulab

yasulab Jul 13, 2017

Member
  • いい雰囲気だったので突発的に動画を撮ってしまった 📹 👀
Member

yasulab commented Jul 13, 2017

  • いい雰囲気だったので突発的に動画を撮ってしまった 📹 👀
@yasulab

This comment has been minimized.

Show comment
Hide comment
@yasulab

yasulab Jul 13, 2017

Member

@kou PR投げました! Review お願いします 🙏

Put link to YouTube Channel on footer
oss-gate/oss-gate.github.io#47

Member

yasulab commented Jul 13, 2017

@kou PR投げました! Review お願いします 🙏

Put link to YouTube Channel on footer
oss-gate/oss-gate.github.io#47

@yasulab

This comment has been minimized.

Show comment
Hide comment
@yasulab

yasulab Jul 13, 2017

Member
  • One more PR sent to oss-gate
Member

yasulab commented Jul 13, 2017

  • One more PR sent to oss-gate
@yasulab

This comment has been minimized.

Show comment
Hide comment
@yasulab

yasulab Jul 13, 2017

Member

Tweeted. 🐦 https://twitter.com/yasulab/status/885474962995257345

たまたま三脚とカメラと収録機材一式をもっていたので、今日の OSS Gate ミートアップの雰囲気を動画で撮影してみました! 📹 #oss_gate
https://www.youtube.com/watch?v=vjT2BbHzlUM

Member

yasulab commented Jul 13, 2017

Tweeted. 🐦 https://twitter.com/yasulab/status/885474962995257345

たまたま三脚とカメラと収録機材一式をもっていたので、今日の OSS Gate ミートアップの雰囲気を動画で撮影してみました! 📹 #oss_gate
https://www.youtube.com/watch?v=vjT2BbHzlUM

@kou

This comment has been minimized.

Show comment
Hide comment
@kou

kou Jul 13, 2017

Member

たまたますごい!

Member

kou commented Jul 13, 2017

たまたますごい!

@satoryu

This comment has been minimized.

Show comment
Hide comment
@satoryu

satoryu Jul 13, 2017

Member

@yasulab 動画公開、問題ないです! 👍

Member

satoryu commented Jul 13, 2017

@yasulab 動画公開、問題ないです! 👍

@yasulab

This comment has been minimized.

Show comment
Hide comment
@yasulab

yasulab Jul 13, 2017

Member

ありがとうございます! 😸

Member

yasulab commented Jul 13, 2017

ありがとうございます! 😸

@koic

This comment has been minimized.

Show comment
Hide comment
@koic

koic Jul 13, 2017

Member

メンテナンスしている Ruby のライブラリへの frozen string literal 対応をしていたり、Ruby ライブラリのプロジェクトへのパッチの送り方など伝えたりしていました。

Member

koic commented Jul 13, 2017

メンテナンスしている Ruby のライブラリへの frozen string literal 対応をしていたり、Ruby ライブラリのプロジェクトへのパッチの送り方など伝えたりしていました。

@koic

This comment has been minimized.

Show comment
Hide comment
@koic

koic Jul 13, 2017

Member

お疲れ様でした。次回の OSS Gate 東京ミートアップは 8/28 (月) にてお願いします。Doorkeeper の用意まで少々お待ち下さいませ。

Member

koic commented Jul 13, 2017

お疲れ様でした。次回の OSS Gate 東京ミートアップは 8/28 (月) にてお願いします。Doorkeeper の用意まで少々お待ち下さいませ。

@koic koic closed this Jul 13, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment