ember-learn / ember-learn/ember-cli-addon-docs

Clean up `name` argument usage in docs-demo component

オープン
#183 コメント 9 件 リアクション 1 件 担当者 0 名 GitHub で見る
主要言語
JavaScript
スター
172
フォーク
142
PR マージ指標
30日以内にマージされた PR はありません

説明

Couple of issues I'm having with the `name` argument for the docs-demo:

1. I get `Attempted to register a view with an id already in use: example-undefined` a lot, because I forget to add a name. If a name is required, we should assert that it exists when the component is rendered.

2. It seems very verbose to require a name, and then (generally) require each snippet to set its names as well. Could we use the `name` argument to infer the snippets for the demo? Possibly allowing us to just pass in `type` to each snippet?

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

まず docs-demo コンポーネントを見つけ、name 引数と snippet 名の現在の処理を読んでください。要求された 2 つの動作、つまり名前がないことをアサートする動作と、snippet 名または型を推測する動作を比較し、両方のケースが完了していることを示す、サポート対象の使用方法とテストを定義してください。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
javascript
領域
documentation, frontend
issue の種類
機能追加
難易度
4/5
見積もり時間
3〜5日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
35/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。