Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Activity
    Yuki Takei
    @yuki-takei

    その後 app.js などで使うことを明示してあげる? (考え中

    ここの部分でのなんからのこだわりや懸念点がある場合、今回のPRと方向が違ってしまうということが起こりうると思います。例えば plugin.js 内で require して webpack でコンパイルして layout ファイル内の <script> でロードするのが今回の方式ですが、そうではなく SystemJS とか webpack2 以降の import() で動的インポートする方式にしたい、とか。
    ただ、今そういった動くスニペットは存在しないわけですから(本体の webpack も1系だし)、僕としてはまずはお試しで今回のPRを取り込んでいただいて、デベロッパーとして使い勝手を検証しながら機構の善し悪しのジャッジをするフェーズに入れたらなあと思っています。

    Sotaro KARASAWA
    @sotarok
    なるほどー
    いや、言いたいことはわかります
    ただまあ、plugin機構は plugin が色々作られ始めると機構自体を変更するのは結構難しいのでちゃんと考えたいとは思ってます。
    okonomi
    @okonomi
    hexoのプラグイン機構いいですね。
    @yuki-takeiさんの書き込みを見て今更気づきましたがgrunt(やgulpやwebpackとか)もプラグインのしくみありますね。
    内部実装については知見がなくてこれといった意見を持ち合わせてないですが、使う側としてはnpm install crowi-plugin-xxのあと設定に有効にするプラグインを追記とかがやっぱりわかりやすくていいなーと思います。
    okonomi
    @okonomi
    (わかりやすい、というのは、著名なプロダクトと雰囲気が似ていて取っつきやすそう、という意味です)
    okonomi
    @okonomi
    別件ですが、ブックマークボタンをReactコンポーネントにしました。 crowi/crowi#190
    Crowi.apiPost()がなんかパラメータがおかしくなるので合わせて修正しています。
    Sotaro KARASAWA
    @sotarok
    おおー
    ありがとうございます!
    ちなみに
    これ見ていただきたいのですが、
    なんか、最近思ってたのが、
    Component の DOM が bind するまでの一瞬のチラツキがきになるな、と。
    React 化進めてくと DOM 構築が非同期になるので、
    サーバーサイドレンダリングよりもちらついて遅く見えるなと
    それでやってみたのが、これ↑
    事前に、React Component が入るべき場所に、同じアイテムを入れておく (form input とか font icon とか
    そうすると、bind するときにスっと有効になるかんじになって、ちらつく、というかレンダリングが遅いような雰囲気が抑えられるな~と
    なので、
    ここも、★は事前に埋めておきたい、みたいな!
    あとはこれはちょっと元々のHTMLの問題ですが、ここの header 部分、いまいちですよね ... ★1つなのに変更箇所が4箇所っていう ... うう
    (okonomiさんのせいではありません、ぼくのせいです
    Sotaro KARASAWA
    @sotarok
    あれー、なんかコレどういう変更なんだろ。
    ちゃんと見れてないですが、apiPost のほうは params のままで、apiGet のほうは {params} にするのか
    なんでこれがこうなるのか
    https://github.com/crowi/crowi/pull/190/files#diff-f168a84512aebbebe44ea7d6504f8fd1L121
    okonomi
    @okonomi
    @sotarok 反応遅れてすみません!
    レンダリングのちらつきはこちらも気になってました。
    こういうもんなのかな…とか思ってましたが、そういう対策があるんですね。
    ちょっとやってみます。
    apiPostのほうの変更は、こちらもきちんと確認できてないです…
    とりあえず動いたのと、原因を確かめようとするとaxiosに潜っていく必要がありそうだったのでいったん投げちゃった感じです。
    Sotaro KARASAWA
    @sotarok
    yarn 化するか ...
    Norio Suzuki
    @suzuki
    yarn をお試しした時には Node のバージョンとの整合性でアラート出てた気がする(確か reveal.js かな)
    Sotaro KARASAWA
    @sotarok
    脱 reveal もしたいw
    Norio Suzuki
    @suzuki
    そのチケットの存在には気づいてたので、 yarn より先に脱 reveal なのかな、と。
    Sotaro KARASAWA
    @sotarok
    そうしよう
    Sotaro KARASAWA
    @sotarok

    @yuki-takei さんのこれすさまじく進化しててすごいっす
    https://github.com/weseek/crowi-plus

    よいとこ取り込みたいなあー plugin 周りとかですよねえ (ジャッジ遅くてすいません色々

    Sotaro KARASAWA
    @sotarok
    @okonomi
    axis の api が get と post で違う件、たしかにわかって、
    https://github.com/crowi/crowi/pull/208/files#diff-f168a84512aebbebe44ea7d6504f8fd1L121
    ↑こんなかんじで対応しました
    あと、塩漬けしているあいだに React 15.3 → 15.5 になってたこともあって、上記 PR で対応してる … !
    のでご確認いただければとっ
    (問題点は、PropTypes くらい)
    okonomi
    @okonomi
    @sotarok おお、ご連絡ありがとうございます、了解です!
    Sotaro KARASAWA
    @sotarok
    @abouthiroppy crowi/crowi#241
    ですが、
    https://twitter.com/__gfx__/status/876612734766399495
    こんなやりとりがあったのでしたっ
    個人的には TS アリだとおもってるので
    Sotaro KARASAWA
    @sotarok
    @suzuki It can not be changed to that mail address のやつ、バグってることがわかった。なにも変更せずに post したとき、language の変更とかができない (つまり自分の今設定されてるemail addressすら duplicate と判定される)
    Norio Suzuki
    @suzuki
    おっと、確認します
    Sotaro KARASAWA
    @sotarok
    :bow:
    Sotaro KARASAWA
    @sotarok
    @abouthiroppy @suzuki 1.6.3 merge します
    あっ ..
    #259 がまだあった
    Sotaro KARASAWA
    @sotarok
    これ、ちょっと気になることがあるとすると、 $ne をつかってるから無駄に scan が走る気が .. (mongodb の not in の scan ってどうなってるんだろう)
    @suzuki
    ただ、ロジックとしては問題なかったです (thanks
    Yuta Hiroto
    @hiroppy
    gitter見てなかったです😇
    okonomi
    @okonomi
    ご無沙汰しております。
    だいぶ停滞してしまってましたが、またフロント周りをちょっとずつ触っていこうと思います。
    okonomi
    @okonomi
    @sotarok npmがちょっと遅かったりnpm shrinkwrapがなぜかエラー出たりとかつらかったので試しにyarnにしてみようと思ったら意外とすんなりできたんですが、
    https://github.com/okonomi/crowi/compare/master...yarn
    yarnに変更するのってどう思われますか?
    okonomi
    @okonomi
    npm@5へのアップグレードが進行してるのに気づいてませんでした…お騒がせしました :bow:
    crowi/crowi#267
    Sotaro KARASAWA
    @sotarok
    @okonomi こっちで mention してもらってたの全然きづいてなかったです (2ヶ月)
    すいませんすいません
    (だいぶ停滞してたんですが、ようやくいろいろ動き出せる感じになってきました)