Skip to content

Removed that the task template is configured in the group configuration - #1175

Merged
derklaro merged 1 commit into
CloudNetService:nightlyfrom
0xIO32:nightly
Apr 9, 2023
Merged

derklaro merged 1 commit into
CloudNetService:nightlyfrom
0xIO32:nightly

Conversation

@0xIO32

@0xIO32 0xIO32 commented Mar 25, 2023

Copy link
Copy Markdown
Contributor

Motivation

We found in the group configuration that it contains the template of the tasks (created on the initial setup). I asked on the Discord if this were a bug, because the template were already on the task configuration and that it shouldn't be there was confirmed by: 0utplay#0187 ( https://discord.com/channels/325362837184577536/818777626663321671/1089277611378753597 )

Modification

I've modified the DefaultTaskSetup class to not include the template of the task to the group

Result

Now the template of the tasks (created on the initial setup) will not be inside the group configuration.

Other context

I've added "mlunkeit" as co-contributor in the commit because we discovered it together.
Fixes #

Co-authored-by: mlunkeit <malte.lunkeit@gmail.com>
@derklaro
derklaro merged commit 8b53ca6 into CloudNetService:nightly Apr 9, 2023
derklaro pushed a commit that referenced this pull request Apr 17, 2023
…s well (#1175)

### Motivation
We found in the group configuration that it contains the template of the
tasks (created on the initial setup). I asked on the Discord if this
were a bug, because the template were already on the task configuration
and that it shouldn't be there was confirmed by: 0utplay#0187 (https://discord.com/channels/325362837184577536/818777626663321671/1089277611378753597)

### Modification
I've modified the DefaultTaskSetup class to not include the template of
the task to the group

### Result
Now the template of the tasks (created on the initial setup) will not be
inside the group configuration.
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