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

imprv: Enable marp from adminsetting #7939

Conversation

reiji-h
Copy link
Contributor

@reiji-h reiji-h commented Aug 1, 2023

概要

  • Marp を管理者画面から有効化するオプションを追加する
  • 管理者画面で無効化していると、Frontmatter で marp: true を指定しても使えない

Task

Screenshot

スクリーンショット 2023-08-04 121332

@reiji-h reiji-h changed the title Imprv/115673 115675 enable from adminsetting imprv: enable marp from adminsetting Aug 1, 2023
@reiji-h reiji-h marked this pull request as draft August 1, 2023 09:41
Copy link
Contributor

@jam411 jam411 left a comment

Choose a reason for hiding this comment

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

  • /workspace/growi/apps/app/src/server/models/config.ts で isEnabledMarp の初期値を設定してほしいです。

  • 管理画面の配置とデザインを dev-uiux で yuki さん fumiya-s さん shintaro-o さんにメンションしてこんな感じでやろうと思っていること垂れ流しておいてほしいです。ツッコミがあれば教えてくれると思う。武井さんは review のタイミングでもいいかもしれないが。

apps/app/src/pages/me/[[...path]].page.tsx Outdated Show resolved Hide resolved
apps/app/src/pages/_search.page.tsx Outdated Show resolved Hide resolved
apps/app/src/client/services/AdminCustomizeContainer.js Outdated Show resolved Hide resolved
apps/app/src/pages/[[...path]].page.tsx Outdated Show resolved Hide resolved
Copy link
Contributor

@jam411 jam411 left a comment

Choose a reason for hiding this comment

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

  • /trash ページと検索ではレンダリングしないで了解です
  • レンダリングする next pages は動作確認で網羅できるようにしておきたいです。忘れずに動作確認タスクに加えておいてください

"select_search_scope_children_as_default_desc": "OFFの場合、検索範囲のデフォルト設定は「全てのページ」になります。"
"select_search_scope_children_as_default_desc": "OFFの場合、検索範囲のデフォルト設定は「全てのページ」になります。",
"enable_marp": "Marp を有効化する",
"enable_marp_desc": "OFFの場合、Marp は使用できません。",
Copy link
Member

Choose a reason for hiding this comment

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

プレゼンテーション表示に Marp を利用できるようになります。

に変更して。

更に、XSS に対して脆弱になり得るということを追記してください。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

次のように変更しました。

image

</p>
</CustomizeFunctionOption>
</div>
</div>
Copy link
Member

Choose a reason for hiding this comment

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

Functions の中ごちゃごちゃしてきたから、「プレゼンテーション」っていうサブセクションがほしいな
後続ストーリーにして meiri-k にアサインしてください。

@reiji-h reiji-h changed the base branch from feat/115673-marp to imprv/115672-presentation-preview August 7, 2023 03:15
@reiji-h reiji-h changed the title imprv: enable marp from adminsetting imprv: Enable marp from adminsetting Aug 7, 2023
@reiji-h reiji-h marked this pull request as ready for review August 7, 2023 03:17
@jam411 jam411 merged commit 5623ca1 into imprv/115672-presentation-preview Aug 7, 2023
@jam411 jam411 deleted the imprv/115673-115675-enable-from-adminsetting branch August 7, 2023 03:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants