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

整理: manage_library feature flag を削除 #1254

Closed
wants to merge 7 commits into from

Conversation

tarepan
Copy link
Contributor

@tarepan tarepan commented May 16, 2024

内容

概要: 役割を終えた manage_library feature flag を削除するリファクタリングを提案します。

現在の engine_manifest.jsonmanage_library.valueTrue である。
かつてはこれが False であり、generate_app() とルーター内でこの値を参照する feature flag により実装を無効化していた。
ゆえに現在、この feature flag は役割を終えている。不要な feature flag は削除が望ましい。

このような背景から、manage_library feature flag を削除するリファクタリングを提案します。

関連 Issue

無し

@tarepan tarepan requested a review from a team as a code owner May 16, 2024 07:39
@tarepan tarepan requested review from Hiroshiba and removed request for a team May 16, 2024 07:39
Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

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

あ、ライブラリインストール機能がなかった場合にAPIが使えると思われないようにしたいという意図もあります🙇
(sing実装時にsing系もそうすべきだったかもと今気づきました)
申し訳ないのですがそのままでお願いできると・・・🙇

ちなみに製品版ではOFFにしてたりします・・・・・・と思ってたのですが、製品版もtrueになってしまっていますね。。。

関係ないのですが、404返す部分はDependsを使えばコピペ部分減らせるかも・・・?

@tarepan
Copy link
Contributor Author

tarepan commented May 28, 2024

製品版もtrueになってしまっています

こちらを確認して「役割を終えている」と判断した次第です。バグとのことなので PR close とします。

404返す部分はDependsを使えばコピペ部分減らせるかも

👍️
減らせそうです。

@tarepan tarepan closed this May 28, 2024
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.

2 participants