Skip to content

Add local property to elements#52

Merged
goetas merged 1 commit intogoetas-webservices:masterfrom
Toilal:element-local-property
Aug 22, 2020
Merged

Add local property to elements#52
goetas merged 1 commit intogoetas-webservices:masterfrom
Toilal:element-local-property

Conversation

@Toilal
Copy link
Copy Markdown
Contributor

@Toilal Toilal commented Feb 12, 2020

@goetas
Copy link
Copy Markdown
Member

goetas commented Jun 24, 2020

Could you please simplify a bit this XMLs used for tests, reducing them to the minimum needed? I think that all those props are not needed. It will make those tests more readable and maintainable in the future.

@Toilal Toilal force-pushed the element-local-property branch 2 times, most recently from a9ea3c2 to 59efafc Compare August 21, 2020 22:45
@Toilal Toilal force-pushed the element-local-property branch from 59efafc to 47ceba4 Compare August 21, 2020 22:47
@Toilal
Copy link
Copy Markdown
Contributor Author

Toilal commented Aug 21, 2020

@goetas I have reduce the tests to the minimum. I hope you will be able to merge soon now 👍

@goetas goetas merged commit 5235c95 into goetas-webservices:master Aug 22, 2020
@goetas
Copy link
Copy Markdown
Member

goetas commented Aug 22, 2020

Thanks for your contribution

@Toilal Toilal deleted the element-local-property branch August 24, 2020 08:32
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