負債を減らした新人の話

Slide 1

Slide 1 text

負債を減らした新人の話 minimo事業部プロダクト開発G 開発T 萩原涼介 Dive into mixi night! #3 2018.08.22

Slide 2

Slide 2 text

minimo 使ってる人

Slide 3

Slide 3 text

ありがとうございます！

Slide 4

Slide 4 text

自己紹介

Slide 5

Slide 5 text

萩原涼介 - Hagihara Ryosuke Name 萩原涼介 - Hagihara Ryosuke HN らりょす (raryosu) 所属 minimo事業部業務サーバサイド・Webフロントエンド出身校津山工業高等専門学校出身広島県福山市生年月日 1997年１０月７日（２０歳）

Slide 6

Slide 6 text

No content

Slide 7

Slide 7 text

No content

Slide 8

Slide 8 text

No content

Slide 9

Slide 9 text

No content

Slide 10

Slide 10 text

このプレゼンの内容

Slide 11

Slide 11 text

はなすこと・はなさないことはなすことはなさないこと・ミクシィの新卒エンジニアのはたらきかた・むずかしい技術の話・非公開の情報

Slide 12

Slide 12 text

minimoのプロダクト

Slide 13

Slide 13 text

minimo の展開スマホアプリ iOS: Obj-C, Swift Android: Java, Kotlin SALON TOOL React.js Webアプリ Perl (Mojolicious) 掲載者向けオウンドメディア裏方担当 WordPress

Slide 14

Slide 14 text

minimo の展開スマホアプリ iOS: Obj-C, Swift Android: Java, Kotlin SALON TOOL React.js Webアプリ Perl (Mojolicious) 掲載者向けオウンドメディア僕がメインで担当裏方担当 WordPress

Slide 15

Slide 15 text

minimoの掲載について

Slide 16

Slide 16 text

minimoへの掲載掲載者掲載を作成お客さん掲載を閲覧 minimodel.jp minimoアプリ

Slide 17

Slide 17 text

掲載の公開設定掲載者掲載を作成お客さん掲載を閲覧 minimodel.jp minimoアプリ is_enable is_open

Slide 18

Slide 18 text

どうしてこうなっている？

Slide 19

Slide 19 text

Web検索に引っかかると  困る掲載者さんもいるお店にナイショでminimoをやってる人もいる

Slide 20

Slide 20 text

何が問題だったか

Slide 21

Slide 21 text

is_open 問題 CSから下記のような問い合わせが多くなっていると相談を受けた ⇒ Webの公開設定を分けることによる不都合のほうが多くなってしまった

Slide 22

Slide 22 text

対応の方針 minimodel.jpでの公開設定もアプリでの掲載設定と一致させる

Slide 23

Slide 23 text

既存掲載の対応方法案 1. DB上の is_enable=0 & is_open=1 のものを is_open=0 に書き換える 2. Controller で is_enable=0 のものを呼び出されたときに404を返す 3. is_open を参照せずに is_enable フラグだけを使うようにする

Slide 24

Slide 24 text

対応方法案 1. DB上の is_enable=0 & is_open=1 のものを is_open=0 に書き換える 2. Controller で is_enable=0 のものを呼び出されたときに404を返す 3. is_open を参照せずに is_enable フラグだけを使うようにする

Slide 25

Slide 25 text

どうやって進めていくか

Slide 26

Slide 26 text

当初の考え実装リリース

Slide 27

Slide 27 text

実際仕様変更による影響について関係部署と打ち合わせリリース実装方針のすり合わせ（アプリ・Web・サロンツール・サーバサイド）実装（苦難の連続）リリースタイミングの調整

Slide 28

Slide 28 text

関係部署との打ち合わせ仕様を追加・削除・変更する ⇒ CSや営業担当にも影響が及ぶことも…！・本当にその仕様をなくしていいか  ・なくなることで困るユーザさんはいない？  ・想定されるお問い合わせの内容共有・対応相談  ・掲載者への案内の方法今回の場合

Slide 29

Slide 29 text

プロダクト開発担当チームとの打ち合わせ・ディレクター・エンジニアのトップ・アプリエンジニア・サーバサイドエンジニア・サロンツールを作っているエンジニア・ minimodel.jp や管理ツール・掲載代行ツールを作っているエンジニア（ぼく）

Slide 30

Slide 30 text

プロダクト開発担当チームとの打ち合わせ・ is_open を廃止して， is_enable を参照するようにする・ is_open の書き換えをできないようにする  ・ Web 公開設定を変更できるバージョンのアプリではエラーを返す・ is_open の値を必要とする場面では is_enable を is_open に擬態して返す  ・これは暫定処置決めたこと

Slide 31

Slide 31 text

Slide 32

Slide 32 text

いざ実装！

Slide 33

Slide 33 text

しかし…

Slide 34

Slide 34 text

一筋縄ではいかない掲載のプレビューを閲覧できる機能で is_open を利用していたサロンツール Android スナップフォトをシェアするかどうかの判断に is_open を利用していた iOS/Android is_enable = 0 から 1 に変えたときも is_open を変更しようとしてエラーが返る

Slide 35

Slide 35 text

仕様調整掲載のプレビューを閲覧できる機能で is_open を利用していたサロンツールうまいこと修正してもらった

Slide 36

Slide 36 text

仕様調整サロンツールうまいこと修正してもらった Android スナップフォトをシェアするかどうかの判断に is_open を利用していた（ is_enable を参照するようにしてもらった）

Slide 37

Slide 37 text

仕様調整サロンツールうまいこと仕様変更した Android is_open を 0 → 1 に変更するときにはうまく行ったように見せかけてなにもしない iOS/Android 掲載を非公開→公開に変えたときも is_open を変更しようとしてエラーが返る

Slide 38

Slide 38 text

実装完了

Slide 39

Slide 39 text

流れの確認MTG @ 7/11

Slide 40

Slide 40 text

リリーススケジュールについて相談 7/17, 18 QAタスクの作成・レビュー・リリース 7/19 リリース営業・CS担当のチームに日程の共有・相談

Slide 41

Slide 41 text

リリース当日 @ 7/19

Slide 42

Slide 42 text

No content

Slide 43

Slide 43 text

No content

Slide 44

Slide 44 text

No content

Slide 45

Slide 45 text

翌朝 @ 7/20

Slide 46

Slide 46 text

No content

Slide 47

Slide 47 text

なんかエラー出た  （ is_open を受け取らないように変更したメソッドに is_open を渡そうとしていた）

Slide 48

Slide 48 text

まとめ

Slide 49

Slide 49 text

仕様をなくす施策を進めるのには勇気がいる新しく付け足すのは比較的簡単でも，すでにあるものをなくしていくのはけっこうたいへん

Slide 50

Slide 50 text

仕様をなくす施策を進めるのには勇気がいるだけど，それによって仕様が把握しやすくなった開発しやすくなった

Slide 51

Slide 51 text

ぼくはまんぞくです

Slide 52

Slide 52 text

新卒に任せられる仕事の幅が広い

Slide 53

Slide 53 text

業界を変えることができるワクワクするプロダクトに関われる

Slide 54

Slide 54 text

No content