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: resolve binding I select the related '(.*)' entity in the lookup #158

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

osagiestar
Copy link
Collaborator

Bug fixed and tested

@osagiestar osagiestar changed the title tes: SelectARelatedEntityInALookup failing #149 fix: resolve binding I select the related '(.*)' entity in the lookup Oct 27, 2022
@tdashworth
Copy link
Contributor

Hi @osagiestar,

What was the fix for the I select the related '(.*)' entity in the lookup binding? It looks to be the same change as #157

@osagiestar
Copy link
Collaborator Author

osagiestar commented Oct 28, 2022

Hi @osagiestar,

What was the fix for the I select the related '(.*)' entity in the lookup binding? It looks to be the same change as #157

@tdashworth : The fix for #157 resolved this bug as well but to test and ensure it (#158) is fixed, I had to remove the "@ignore" syntax in the "RelatedGridSteps.feature". Hence, this PR.

@tdashworth
Copy link
Contributor

@osagiestar it looks like that test is still failing on the build server: https://capgeminiuk.visualstudio.com/GitHub%20Support/_build/results?buildId=8240&view=ms.vss-test-web.build-test-results-tab&runId=1015462&resultId=100157&paneView=debug

I'll try locally now.

@tdashworth
Copy link
Contributor

@osagiestar it looks like that test is still failing on the build server: https://capgeminiuk.visualstudio.com/GitHub%20Support/_build/results?buildId=8240&view=ms.vss-test-web.build-test-results-tab&runId=1015462&resultId=100157&paneView=debug

I'll try locally now.

Interesting, it worked locally...

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