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

Remove use_role requirement to existing JT.project to make changes to JT #14278

Draft
wants to merge 2 commits into
base: devel
Choose a base branch
from

Conversation

AlanCoding
Copy link
Member

SUMMARY

This is an adjustment to #13818

Unfortunately, I don't see any easy choices here. The "sensitive" change requirements are still suffocating.

This accomplishes the main stated goal of the prior PR and issue - which is to change the project while lacking use_role to the existing project. This is accomplished by the subtle obj=None in the call to check_related. It might be apparent, but I'm very opposed to the solution method in the PR 13818 because it adds more logic to an already out-of-control collection of business logic.

ISSUE TYPE
  • New or Enhanced Feature
COMPONENT NAME
  • API

@AlanCoding AlanCoding changed the title Remove requirement for use_role to JT.project for changing stuff Remove use_role requirement to existing JT.project to make changes to JT Jul 26, 2023
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.

2 participants