動態 meta 的 lib

[ohmkao]

https://github.com/5fpro/election2016-official/blob/develop/app/lib/meta_context.rb#L4

  def render *args
    # 包含 namespace 判斷
    # EX1: PostsController#press => meta_for_posts_press
    # EX2: Admin::FlagsController#show => meta_for_admin_flags_show
    method_name = "meta_for_#{controller_path.underscore.gsub(/\//, '_')}_#{action_name}"
    meta_data = self.respond_to?(method_name) ? send(method_name) : meta_for_default
    set_meta(meta_data)
    super
  end

[marsz]

找一天詳細討論一下，大家一起，把現在遇到的問題釐清，再看怎麼解決好了
@phrase5036 @kzdahlia @ohmkao

[marsz]

我需要整理問題的案例列表

[phrase5036]

我跟歐姆約9/5號會一起檢查meta遇到的問題, 因為小英那邊會先refactor一次
剛好可以把遇到的問題list上來 :D
(btw, 如果像是這種告知的訊息, 我們issue還是一樣assign回去給爐主嗎？還是slack通知就可以了？)

[marsz]

僅 comment 不 assign （算是一種 memo 的概念）
等做完了再 assign

btw, 是 10/5 吧?

[phrase5036]

對, 10/5 哭哭, 感謝

[phrase5036]

分兩個地方討論

---

目前的meta_tag_helper遇到的問題
參考 https://github.com/5fpro/rails4-template/blob/with-assets/app/helpers/meta_tag_helper.rb

• 第9行, data[:title] render 出來不會是有包含 site_name 的 title
  • ex :<title>測試 | site_name</title> & <meta content="測試" property="og:title" />
• data[image], 應該也要有預設值, 然後要考量 FB 不吃 "//" 開頭的圖片網址問題
• 假設我今天有個page是不需要有site_name的, 目前的狀況無法拔除, 會影響其他page的meta, 主要是想問, 這種需求合不合理？我們有沒有必要開發這種彈性？

---

歐姆提出的 meta_context.rb 版本
參考 : https://github.com/5fpro/election2016-official/blob/develop/app/lib/meta_context.rb#L4

• 不想要在controller看到meta相關的東西, 所有設定都擺在meta_context.rb
  • 覺得controller還要再多一行set_meta有點多餘
• 依照controller + action 定義 每個 routes 的 meta method
• 目標是把這個構想整合在mars版的meta_tag_helper.rb
• 只要render view 就會自動去抓meta
• 好處 :
  • meta全部控制都在同一隻file
  • controller可以省掉這些set_mata(data)的行數
• 壞處 :
  • 動態寫法會有一些比較稍微複雜的地方

[ohmkao]

修改完成 @phrase5036 幫我看看
已包含 spec 的檢測方式
#35