Re: [PR] fix #3666, fix #3588 (hop)

2024-03-19 Thread via GitHub


sramazzina commented on PR #3673:
URL: https://github.com/apache/hop/pull/3673#issuecomment-2008169747

   Tested on MySQL, PostgreSQL, DuckDB and SQL Server with success. In my 
opinion this PR can be merged


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@hop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] fix #3666, fix #3588 (hop)

2024-02-27 Thread via GitHub


sramazzina commented on PR #3673:
URL: https://github.com/apache/hop/pull/3673#issuecomment-1966010021

   I put the PR in draft waiting to do some more tests with some commonly used 
databases.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@hop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] fix #3666, fix #3588 (hop)

2024-02-26 Thread via GitHub


sramazzina commented on PR #3673:
URL: https://github.com/apache/hop/pull/3673#issuecomment-1965425125

   The show layout dialog must be completely rewritten because the current 
dialog is completely out of context. In any case the issue is now fixed


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@hop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org