Skip to content

Conversation

rie-oki-agn
Copy link
Member

@rie-oki-agn rie-oki-agn commented Jul 2, 2022

#46 😎 /tutorial
(後でtutorial.vue -> simulation.vue に変更)

@rie-oki-agn rie-oki-agn linked an issue Jul 2, 2022 that may be closed by this pull request
@netlify
Copy link

netlify bot commented Jul 2, 2022

Deploy Preview for commit-mate-net ready!

Name Link
🔨 Latest commit f9a2b3b
🔍 Latest deploy log https://app.netlify.com/sites/commit-mate-net/deploys/62bfd2210dc0230008b7ea8f
😎 Deploy Preview https://deploy-preview-48--commit-mate-net.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@rie-oki-agn rie-oki-agn changed the title issue#46 GitHub Simulation App ( PR for Netlify preview) issue brach 46-simulation GitHub Simulation App ( PR for Netlify preview) Jul 2, 2022
@rie-oki-agn rie-oki-agn changed the title issue brach 46-simulation GitHub Simulation App ( PR for Netlify preview) issue brach 46 GitHub Simulation App (For Netlify preview) Jul 2, 2022
@rie-oki-agn rie-oki-agn changed the title issue brach 46 GitHub Simulation App (For Netlify preview) issue branch 46 GitHub Simulation App (For Netlify preview) Jul 2, 2022
@rie-oki-agn rie-oki-agn changed the title issue branch 46 GitHub Simulation App (For Netlify preview) 46 GitHub Simulation App (For Netlify preview) Jul 2, 2022
@wako-p
Copy link
Contributor

wako-p commented Jul 4, 2022

少しコードが複雑&肥大化してきたかなって思ったので
コンポーネント側のコードをシンプルに保つために、GitHubをシミュレーションするロジックを抽出→クラスとしてまとめるサンプルコードを作ってみました。
https://github.com/wako-p/commit-mate-s-github-simulate-app
テストコードなしで書く自信がなかったので私のリポジトリで作成してます。

例えば、このクラスを使えばdoClonedoCommitは以下のように書くことができます。

// リポジトリの定義
origin = ref(Repository.create("origin"));
local = ref(null);

// ~省略~

const doClone = () => {
  local = origin.clone()
}

const doCommit = () => {
  local.commit("commit to local main branch");
}

とりあえずで作っただけなので、コンポーネントの描画に必要なデータを提供できているかはわかりませし、
フロントエンド側での碇石?(@monsat さんが↓に書いて下さっているComposables)みたなものもあると思うのでそちらを優先して頂ければと思います!

@monsat
Copy link
Member

monsat commented Jul 4, 2022

そろそろ Composables にしてよいころですね

@rie-oki-agn
Copy link
Member Author

rie-oki-agn commented Jul 7, 2022

@wako-p
おーーー!!すごい、ありがとうございます!
せっかくNuxtなのでcomposablesに書き換えたいですね!!
作ってくださったclass構文のソースを参考にcomposablesにしてみようかと思います!
今週末までに作ってみますね!

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.

GitHub Simulation App
3 participants