Skip to content
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

チュートリアル上でファイル操作を伴う課題を修正 #55

Merged
merged 5 commits into from
Jul 29, 2024

Conversation

aster-mnch
Copy link
Collaborator

@aster-mnch aster-mnch commented Jul 29, 2024

Fix: #54

変更内容

現在のチュートリアル機能では、ブラウザ上でファイル操作 (ファイルやフォルダの作成) を行うことができないため、いくつかの章のチャレンジで実施できないものがあります。

理想的には StackBlitz のようなファイル操作ができることですが、適用までに時間がかかることから、とりあえずチュートリアルの内容をファイル追加などが不要な操作になるように変更しました。
具体的には、元々はユーザーに作成してもらうつもりのファイルを事前に作成しておき、それに合わせてチュートリアルの文言も調整しています。

@aster-mnch
Copy link
Collaborator Author

oops... 今回の変更とは関係ないところで CI が落ちてるな...
先にそっち対応の PR 作るか

@aster-mnch aster-mnch changed the title WIP チュートリアル上でファイル操作を伴う課題を修正 チュートリアル上でファイル操作を伴う課題を修正 Jul 29, 2024
@aster-mnch aster-mnch requested review from ubugeeei and shiyuu33 and removed request for ubugeeei and shiyuu33 July 29, 2024 09:52
@ubugeeei
Copy link
Member

ん、なんかまだこけるな

Copy link
Member

@ubugeeei ubugeeei left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@aster-mnch

一旦マージ、問題部分は後で直しましょう!

@ubugeeei ubugeeei merged commit 8050ebe into main Jul 29, 2024
1 of 2 checks passed
@ubugeeei ubugeeei deleted the remove-file-operations-instructions branch July 29, 2024 10:04
@aster-mnch
Copy link
Collaborator Author

@ubugeeei

あー今見たら nr prepare 内でやってる install で template/basic/pnpm-lock.yml が更新されてるっぽいですね

$ jq .scripts.prepare package.json
"nuxt prepare && pnpm -C ./templates/basic install && pnpm -C ./templates/basic prepare"

$ git st
On branch main
Your branch is up to date with 'origin/main'.

Changes not staged for commit:
  (use "git add <file>..." to update what will be committed)
  (use "git restore <file>..." to discard changes in working directory)
        modified:   templates/basic/pnpm-lock.yaml

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.

チュートリアル上でファイル操作を伴う課題を修正
2 participants