-
Notifications
You must be signed in to change notification settings - Fork 0
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
__dir__の呼び出しをやめる #9
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
見ました! 👀 細かいですが,1点記載させていただきました.
リファインメントしたりスプリントに積めるように, https://insidemine.agileware.jp/redmine/issues/90228 のようなIssueの作成をお願いします 🙇
Issueの記載内容に関連しますが,対応前(master)についてこのプラグインとプラグインを動かすのに必要最低限のプラグインを入れた状態では,エラーなくZeitwerkによる再読み込みができたでしょうか.
チケットを作成しました。
// https://github.com/agileware-jp/redmine_logs/blob/develop/assets/stylesheets/logs.css
#redmine_log_show {
...
# https://github.com/agileware-jp/redmine_logs/blob/develop/app/controllers/logs_controller.rb#L25
class LogsController < ApplicationController
...
include RedmineLogs::LogFile
|
ありがとうございますー
こちらもご確認ありがとうございますー |
Redmine6.0 対応のほうで production code の修正があるので、そちらのブランチにマージしてしまおうと思います |
Story: