Skip to content

Conversation

DelaunayAlex
Copy link
Contributor

@DelaunayAlex DelaunayAlex commented Aug 7, 2024

WARNING Merge after #141

@DelaunayAlex DelaunayAlex self-assigned this Aug 7, 2024
@DelaunayAlex DelaunayAlex marked this pull request as draft August 7, 2024 10:44
@DelaunayAlex DelaunayAlex added the enhancement New feature or request label Aug 7, 2024
@DelaunayAlex DelaunayAlex force-pushed the feature/create_dataset_modal_exec branch 3 times, most recently from 0b9add9 to c7f9f43 Compare August 11, 2024 23:59
@DelaunayAlex DelaunayAlex force-pushed the feature/create_dataset_modal_exec branch 5 times, most recently from fc3d77c to 728b625 Compare August 22, 2024 15:02
@DelaunayAlex DelaunayAlex marked this pull request as ready for review August 26, 2024 15:06
@DelaunayAlex DelaunayAlex force-pushed the feature/create_dataset_modal_exec branch 2 times, most recently from 9fa1c37 to 58ad3a6 Compare August 27, 2024 09:08
Copy link
Member

@nbrouand nbrouand left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Your formatter remove space in each import. Could you fix it please ?

I click edit without changing anything, but it says custom dataset. Do you think it's correct ?
Don't you think it's clear that when we save, we do not override current dataset values ? SHould we add another checkbox to override updated dataset ?

I cannot save a dataset anymore
image

And i cannot edit dataset, i am always redirect on this url :
https://localhost:4200/#/dataset/undefined/edition

Copy link
Member

@nbrouand nbrouand left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The message " Le nom du dataset est vide " should appear only on click on "execute"

The dataset name input could be wider now

When i am on campaign history :
image

When i replay last campaign execution :
image

@nbrouand
Copy link
Member

Use constants and datable from report instead of store them in database

@DelaunayAlex DelaunayAlex force-pushed the feature/create_dataset_modal_exec branch 3 times, most recently from e8f884f to 30891ab Compare September 3, 2024 12:12
@nbrouand
Copy link
Member

nbrouand commented Sep 3, 2024

image
show errror message on click on execute not directly when we click inside the intput

@nbrouand
Copy link
Member

nbrouand commented Sep 3, 2024

do not enable execute button while loading environment because if we are too fast we can click before the component is fully loaded and it leads to an error.

@nbrouand
Copy link
Member

nbrouand commented Sep 3, 2024

when I click on "new dataset"
image

@DelaunayAlex DelaunayAlex force-pushed the feature/create_dataset_modal_exec branch from 41928b8 to c85e7df Compare September 11, 2024 14:10
@boddissattva
Copy link
Member

When i execute a scenario with a named dataset, i got Custom as dataset name in report.
image
image

@boddissattva
Copy link
Member

On campaign edition, strange error message
image

@boddissattva
Copy link
Member

If we allow this
image
There could be some clash in campaign report !! (but scenario executions are ok)
image
image
image

On campaign edition, if a scenario is selected multiple times, every selection has to have a dataset set.

@boddissattva
Copy link
Member

In campaign report export, we have
image

Could be good to keep the name for the moment ? And add a task for adding the dataset content used.

@boddissattva
Copy link
Member

boddissattva commented Sep 12, 2024

When i click on a dataset filter value, it unchecks itself automagically !! So can't select a dataset filter at all.

@nbrouand
Copy link
Member

Bug:
create a campaign with a dataset
save
update the campaign by removing the dataset
run => error
image

@boddissattva
Copy link
Member

boddissattva commented Sep 16, 2024

I got some executions on a given scenario
image
When i set a default dataset on this scenario, all the old executions without dataset show the use of the default dataset
image

Do we have to keep the default dataset declaration on scenarios ?

When i put this scenario (with a default dataset) in a campaign (without a default dataset) and execute the campaign manually without a dataset, the scenario default dataset is not used.
The same thing happens for a planified campaign execution.

Note :

  • The dataset filter seems to filter correctly the scenario's executions.
  • When i execute a campaign (with default dataset) manually without a dataset, the default dataset is used.

@nbrouand
Copy link
Member

bug: import/export on dataset not present anymore
image

@DelaunayAlex DelaunayAlex force-pushed the feature/create_dataset_modal_exec branch from 8179574 to fe05762 Compare September 17, 2024 07:30
@boddissattva
Copy link
Member

When i save a campaign, i got an error
image

@nbrouand nbrouand merged commit 2bcbd70 into main Sep 17, 2024
3 checks passed
@nbrouand nbrouand deleted the feature/create_dataset_modal_exec branch September 17, 2024 14:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants