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: AI/GS Reinstate custom settings profile. #7506

Closed
wants to merge 1 commit into from

Conversation

SamuXarick
Copy link
Contributor

No description provided.

@PeterN
Copy link
Member

PeterN commented Apr 13, 2019

Why?

@TrueBrain
Copy link
Member

Owh, @SamuXarick , we talked about this. No more Pull Requests without giving context WHY we would want this. @PeterN is being nice. I am not.

I don't see how this is useful; given there is no further reasoning, closing this topic. You are not allowed to bring up the same topic or discuss "why did you close this PR" for at least 6 months.

@TrueBrain TrueBrain closed this Apr 13, 2019
@SamuXarick
Copy link
Contributor Author

Because it is a requirement when creating settings for an AI/GS. Why make it a requirement if then it's not accessible?

@PeterN
Copy link
Member

PeterN commented Apr 13, 2019

"Custom" does not make sense in the context it is used. Instead it should probably be removed.

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.

None yet

3 participants