Merged
Conversation
This was referenced Mar 29, 2025
rand-honda
force-pushed
the
rcos-release_20250328
branch
13 times, most recently
from
d9875d7 to
4c41eed
Compare
rand-honda
force-pushed
the
rcos-release_20250328
branch
2 times, most recently
from
09c0af0 to
9bf323d
Compare
hide24
requested changes
Apr 14, 2025
| return None, None | ||
|
|
||
| def download_file(url, cookies): | ||
| response = requests.get(url, cookies=cookies).content |
Author
There was a problem hiding this comment.
download_fileの呼び元でファイルサイズチェックを実施しておりますので、
問題ないかと思います。
Comment on lines
27
to
28
| max_file_size = 128 # MB | ||
| # max_file_size = 128 # MB | ||
|
|
||
| # MAX_UPLOAD_SIZE is defined in bytes, but max_file_size must be defined in MB | ||
| max_file_size = MAX_UPLOAD_SIZE // (1024 ** 2) | ||
|
|
There was a problem hiding this comment.
これはdefaults.py の記述が間違っているのですから、defaults.pyの方が修正されるべきです。
また、これは既存のバグなので別途課題として起票されるのがより適切だと考えます。
Author
There was a problem hiding this comment.
マージ元のプロトタイプソースをそのまま適用しておりました。
修正意図や仕様を把握していませんので。
取り急ぎ修正前にもどしました。
There was a problem hiding this comment.
locked の実装について、テストを拡充してください。
既存の機能に対して悪影響はありませんか。
api/base/settings/defaults.py
Outdated
Comment on lines
460
to
461
| TIME_STAMP_AUTHORITY_URL = 'https://www.langedge.jp/tsa' | ||
| TIME_STAMP_AUTHORITY_URL = 'http://eswg.jnsa.org/freetsa' |
There was a problem hiding this comment.
これは本件開発に関連した更新ですか?
別件の更新が混入しているように見えます。
There was a problem hiding this comment.
以前にあったフィールド名の命名規則についての指摘が反映されていないようです。(get_の件)
他の関連ファイルを含めて確認ください。
rand-honda
force-pushed
the
rcos-release_20250328
branch
4 times, most recently
from
e69b776 to
bfbce2f
Compare
rand-honda
force-pushed
the
rcos-release_20250328
branch
from
bfbce2f to
e040f0a
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Purpose
OA加速化ワークフロー_デモ版（RCOS）
デプロイ対象環境：rcos-release
デプロイ希望日：なるべく早めにお願いいたします
以下３つをセットでお願いいたします
#588
RCOSDP/RDM-ember-osf-web#148
RCOSDP/RDM-waterbutler#74
#578
の修正内容も含みます
Changes
■R-OA-01_WEKO連携機能への論文データおよび書誌情報登録フローの追加
・G-M-1 査読付き論文最終稿 と 根拠データのメタデータ登録
・G-M-４ 査読付き論文最終稿 と 根拠データをWEKOに送出する
※以下のソースを使用
https://redmine.devops.rcos.nii.ac.jp/issues/49044#note-14
■R-OA-02_WEKO連携機能への研究データ付随メタデータと書誌情報の登録の統合化
１．管理責任者承認ワークフローの準備
・G-4 ワークフロー操作画面
・G-4-1 ワークフロー管理画面
２．管理責任者承認ワークフローの実行
・G-2 プロジェクトダッシュボード画面
・G-3 ファイルビュー画面
・G-4 ワークフロー操作画面
・G-4-3 ワークフロータスク管理画面
・G-U-7 ワークフロータスク入力画面
・ワークフロー開始時に該当のファイルをロックする機能を追加
https://redmine.devops.rcos.nii.ac.jp/issues/49826
から必要項目を取得した
３．管理責任者の進行状況確認
・G-4-2 ワークフロープロセス管理画面
・G-4-3 ワークフロータスク管理画面
QA Notes
-->
Documentation
Side Effects
Ticket