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

上書きインポート機能の改良 #1002

Merged
merged 4 commits into from
Oct 13, 2023
Merged

上書きインポート機能の改良 #1002

merged 4 commits into from
Oct 13, 2023

Conversation

acutus-fujii
Copy link

以下の 2つの issue に対応します。

同じファイル server/utils/book/importBooksUtil.ts を変更するため、ひとつのプルリクにしています。

#994 の経緯は次のとおりです。

ブックを上書きインポートするとき、自身の著作でないトピックが含まれている場合は、次のようなメッセージが表示されます。

image

@ties-makimura
Copy link

ties-makimura commented Oct 12, 2023

@acutus-fujii

一括登録でパワーポイントファイルをアップロードするとサーバが異常停止する #994

パワーポイントファイルをアップロードしても,サーバ停止は特に起こらず,問題なくそのまま利用できるのを確認できました.
インポート結果が「jsonファイルがありません」と出るのは想定通りでしょうか?

以下は問題ありませんでした.

  • 階層構造を持つ zip ファイルのアップロードも処理されることを確認
    (sample.zip/sample/aaa.json みたいな構造でテスト)
  • 自身が教師の場合,ブック上書き時にトピックに他人の著作が含まれている場合に上書きしないことを確認
  • 自身が管理者の場合,他人の著者でも無視して上書きできるのを確認

@acutus-fujii
Copy link
Author

インポート結果が「jsonファイルがありません」と出るのは想定通りでしょうか?

はい、想定どおりです。PPTX ファイルを zip ファイルだと思って展開した結果、jsonファイルが見つからなかったと言っています。このメッセージは以前からあるメッセージで、jsonファイルを含まない zip ファイルをアップロードしたときにも表示されます。

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.

None yet

2 participants