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

If .al (object) file is in edit mode (not yet saved) TestScriptor cannot not be in sync #47

Open
lvanvugt opened this issue Sep 25, 2020 · 5 comments
Assignees
Labels
version B Next to be picked up (but needs probably some detailing)

Comments

@lvanvugt
Copy link
Contributor

Can we make TestScriptor to ask user to save file first?

@lvanvugt lvanvugt added the version B Next to be picked up (but needs probably some detailing) label Sep 25, 2020
@DavidFeldhoff
Copy link
Collaborator

What do you mean? So the .al file is in a dirty state (=has changes), but on the UI I change some element? And then there should be a question if the user wants to save the file first?

@lvanvugt
Copy link
Contributor Author

Yes!

@DavidFeldhoff
Copy link
Collaborator

Hm.. would be possible. But when should that happen? If he selected the scenario?

I think it would be better if we just get the current (maybe dirty) content of the file on selecting the scenario. And if the user edits the current scenario, then the ATDD could be updated - Then the ATDD is always in sync.

But I don't know how much work that would be for Marton.. but I think that would be the better solution

@DavidFeldhoff
Copy link
Collaborator

Hm.. would be possible. But when should that happen? If he selected the scenario?

I think it would be better if we just get the current (maybe dirty) content of the file on selecting the scenario. And if the user edits the current scenario, then the ATDD could be updated - Then the ATDD is always in sync.

Unfortunately I don't know how much work that would be for Marton. Could be that that change won't be that easy.. but I think that would be the better solution

@lvanvugt
Copy link
Contributor Author

lvanvugt commented Oct 2, 2020

Makes sense.

@martonsagi ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
version B Next to be picked up (but needs probably some detailing)
Projects
None yet
Development

No branches or pull requests

2 participants