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

Add publish_project permission #31

Closed
wants to merge 25 commits into from

Conversation

ishikawa999
Copy link
Contributor

@ishikawa999 ishikawa999 commented Jan 16, 2021

https://www.redmine.org/issues/9029

  • 権限 publish_projectを追加
  • is_publish == falseの場合、publish_project権限がある場合のみしか値を更新できないように(内部的に)
  • 画面側で、権限がない && 現在の設定がis_public == falseの場合に、is_publicのチェックボックスをdisabledにする
  • テストを書く
    • fixtureも更新する
  • Migrationファイルを追加して、add_projectかedit_projectの権限がある場合はpublish_project権限のデフォルト値をtrueにできるようにする

Your contributions to Redmine are welcome!

Please open an issue on the official website instead of sending pull requests.

Since the development of Redmine is not conducted on GitHub but on the official website and core developers are not monitoring the GitHub repo, pull requests might not get reviewed.

For more detail about how to contribute, please see the wiki page Contribute on the official website.

@ishikawa999 ishikawa999 linked an issue Jan 16, 2021 that may be closed by this pull request
kfischer-okarin and others added 20 commits December 16, 2022 10:56
* add postgres 10.21, 11.16, 12.11, 13.7, 14.3
* use versions existing on hub.docker.com (10.20, 11.15, 12.10, 13.6, 14.2)
* Upgrade ci postgres version from 9.5 to 10 (drop 10.20, 11.15, 12.10, 13.6, 14.2)

Co-authored-by: Ko Nagase <nagase@georepublic.co.jp>
* Add VSCode devcontainer用の設定を追加
* Add docker-compose.yml
* Move Dockerfile-for-redmine-dev-mirror to .devcontainer/Dockerfile
* Fix Rubyのバージョン変更の手順
* Update README.md
* Update DockerfileでADDしなくなったことによって機能しなくなっていたコードを削除
@github-actions
Copy link

Patch can be downloaded here

@sanak
Copy link
Member

sanak commented Jan 12, 2023

こちらですが、今日パーミッション名を変更した類似のパッチがマージされたようでした。

こちらとマージされたパッチの差分を見た感じ、パーミッション名の付け方の違いが影響したようにも思いましたが、こちらのPRと #32 の方はクローズできそうな気がしましたので、念のため記載しておきます。

@sanak
Copy link
Member

sanak commented Feb 23, 2023

こちらですが、今日パーミッション名を変更した類似のパッチがマージされたようでした。

こちらとマージされたパッチの差分を見た感じ、パーミッション名の付け方の違いが影響したようにも思いましたが、こちらのPRと #32 の方はクローズできそうな気がしましたので、念のため記載しておきます。

こちら、類似のパッチが本家側でマージされ、コンフリクト解決が実質不可能と思われましたので、勝手ながら #32 と合わせてクローズさせて頂きます。🙇‍♂️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

プロジェクトを公開する権限を追加する
5 participants