Monica-CodingWorld commented on PR #5264:
URL: https://github.com/apache/fineract/pull/5264#issuecomment-3749764195

   > Ádám wrote:
   > 
   > > it is still not working on existing database
   > 
   > I concur. I've got a local repro as well, taking hints from [Ádám's 
description](https://lists.apache.org/thread/s98zcmwkdq8dn3gn9zp78ygqdnq5dwg2) 
and `.github/workflows/verify-liquibase-backward-compatibility.yml`: I'm 
running `gradle devRun` against a blank db, stopping that, switching to 
`fix-FINERACT-2423`, starting the API again, and I'm getting:
   > 
   > ```
   > liquibase.changelog: ChangeSet 
db/changelog/tenant/parts/0072_add_result_and_status_to_command_source.xml::1::fineract
 encountered an exception.
   > liquibase.exception.DatabaseException: ERROR: column 
"processing_result_enum" does not exist [Failed SQL: (0) ALTER TABLE 
public.m_portfolio_command_source RENAME COLUMN processing_result_enum TO 
status]
   > ```
   > 
   > @Monica-CodingWorld when you were testing, did you try this kind of 
smoke/acceptance test? If not, will you?
   
   @meonkeys I’ve already run spotless, checkMain, checkTest, and the 
integration tests locally and those passed. I haven’t specifically run a 
Liquibase smoke/acceptance test yet, but yes now as you suggested i'll try it 
and I’ll update the PR with the results.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to