RCOSDP / RDM-osf.io

Facilitating Open Science
https://rcos.nii.ac.jp
Apache License 2.0
13 stars 24 forks source link

#24251 コアアドオン定義情報の国際化を実装 #235

Closed ivis-kuwata closed 3 years ago

ivis-kuwata commented 3 years ago

Purpose

24251 コアアドオン定義情報の国際化を実装

Changes

ivis-kuwata commented 3 years ago

恐れ入ります。 いただいた指摘について調査したところ、 pybabelのupdate, およびcompile実行時に 対象となるPOファイルを-Dで指定していないことが 原因だと判明しました。

修正の上、feature/ticket-22451-2ブランチより 以下のプルリクエストを発行いたしましたので ご確認をお願いいたします。

https://github.com/RCOSDP/RDM-osf.io/pull/236

hide24 commented 3 years ago

まず、一般にPRに対して変更の要請があった場合はPRを再作成するのではなく、 そのPRの元になっているbranchに必要な修正をcommitするようにしてください。

次に先の指摘で見落していたのですが、PRの送信先はfeature/以下のブランチではなく フォークもとの最新の開発ブランチに向けてください。 今だと nii-mergework-202007 か develop のどちらかになると思います。

fugithora812 commented 3 years ago

失礼いたしました。本ブランチに修正分をコミットさせていただきました。

また現在NIIの担当者様より、検収用ブランチとしてmergework-202007ブランチから feature/nii-rdc-core-addonブランチを作成いただき、そちらをベースに 開発を進めております。 https://redmine.devops.rcos.nii.ac.jp/issues/23011

開発フローについては、前担当の常川先生の他に込山先生にも お知らせしておりますので、ご確認いただければ幸いです。