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

Fix/tao 4840 cat answer required #980

Open
wants to merge 13 commits into
base: develop
Choose a base branch
from

Conversation

eacraft13
Copy link
Contributor

@eacraft13 eacraft13 commented Aug 28, 2017

Bugfix - ACT
Jira: https://oat-sa.atlassian.net/browse/TAO-4840

Requires

This fix is to show/hide allowSkipping/validateResponses options from test authors based on whether the feature is enabled.

@ssipasseuth
Copy link
Contributor

Hi, I am just wondering whether the product own has approved of this change ? I mean that sounds like a legitimate improvement but there is also the case in which people would like to author the test in TAO and export it to run it somewhere else. In this case I can imagine the complaint that one can no longer turn this attribute to true/false because their intention is to edit the test metadata+attribute not necessarily run it. I hope I am making any sense...

@eacraft13
Copy link
Contributor Author

I had created a think tank issue a while back and commented again not to long ago but haven't heard anything yet. I slacked Mark directly today and hopefully he'll review it soon.

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