"Add Sketch" option visible to guest users on other users collections
メンテナーはふだん 3 日以内に返信
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 68/100
- issue の種類
- バグ
- 明瞭さ
- 明確に書かれている
- 活発さ
- 静か
- 技術スタック
- javascript
- 領域
- frontend
調査の方向性
まず、別のユーザーのコレクションページで問題を再現し、次にコレクションページの caret と、スケッチリストのモーダルを開く "Add Sketch" オプションを追跡します。所有していないコレクションではゲストユーザーにオプションもモーダルも表示されなくなり、既存の所有者向けフローは引き続き利用できれば完了です。
索引モデルが issue の本文から書いたものです。
説明
p5.js version
No response
What is your operating system?
Mac OS
Web browser and version
Safari 26.5
Actual Behavior
It is possible to view other users collections via their public URL: https://editor.p5js.org/msawired/collections but as a rule, it is not possible to submit a sketch to collections if user is not the collection owner. However, "Add Sketch" still shows up for guest users. Furthermore, it opens up the sketch list modal and allows user to select a sketch. Ajax call fails with 403 (as expected) but no response is displayed to the user.
Expected Behavior
"Add Sketch" option and modal should not be displayed to guest users until this feature is supported.
Steps to reproduce
Steps:
- Login to Editor
- View another users collection page such as mine
- Click the caret and select "Add Sketch"
- 主要言語
- JavaScript
- スター
- 1.7k
- フォーク
- 1.7k
- 平均マージ
- 5日 8時間
- マージ済み PR（30日）
- 12
環境構築
- Dockerfile または Docker Compose ファイルあり
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
processing/p5.js-web-editor のほかの issue
-
Awaiting Maintainer Approval Bug
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
processing/p5.js-web-editor#4392 · コメント 1 件 ·
メンテナーはふだん 3 日以内に返信
-
Renaming a folder doubles the folder name in file paths (console errors show /lib/lib/helper.js)オープンAwaiting Maintainer Approval Bug
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
processing/p5.js-web-editor#4365 · コメント 1 件 ·
メンテナーはふだん 3 日以内に返信
-
p5 logo is broken on verification emails対応中かも @sourabh65mehta が 15 日前に担当しました。 オープンBug
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
processing/p5.js-web-editor#4364 · コメント 2 件 · 担当者 1 名 ·
メンテナーはふだん 3 日以内に返信
-
`PATCH /editor/project/visibility` returns 200 with `null` and doesn't update when `projectId` is a slug再び着手できるかも @Prbhtsgh が 22 日前に担当しましたが、オープン中のプルリクエストはありません。 オープンAwaiting Maintainer Approval Bug
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
processing/p5.js-web-editor#4348 · コメント 3 件 ·
メンテナーはふだん 3 日以内に返信
-
Fix: example.js stores defaultHTML function reference instead of calling it, causing examples to display raw JavaScript source in preview対応中かも @syedbarkath980 が 27 日前に担当しました。 オープンBug
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
processing/p5.js-web-editor#4344 · コメント 3 件 · 担当者 1 名 ·
メンテナーはふだん 3 日以内に返信
processing/p5.js-web-editor の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
-
factory-active task-identify-harness-labels-done task-identify-issue-type-done
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 66/100
mailgun/mailgun.js#544 ·
-
beginner bug frontend
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
BistaDinesh03/critique#266 ·
メンテナーはふだん 1 日以内に返信
-
New tool: ⚪️ writenode対応中かも @github-actions が今日担当しました。 オープンsubmission triage
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
kausalflow/tools#251 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信