klesh commented on code in PR #2455: URL: https://github.com/apache/incubator-devlake/pull/2455#discussion_r922757550
########## models/migrationscripts/subtask_07112022.go: ########## @@ -0,0 +1,35 @@ +package migrationscripts + +import ( + "context" + commonArchived "github.com/apache/incubator-devlake/models/migrationscripts/archived" + "gorm.io/gorm" + "time" +) + +// Subtask_07112022 DB snapshot model of models.Subtask Review Comment: We added MultiConnection for all other plugins in v0.12. With this `connection` table added, every single table of the plugin was affected, as well as the Domain Layer tables, we have to flush the no-longer-valid data to maintain the data integrity, So, might as well drop existing migration scripts and rebuild the tables. However, the Framework migration should be not affected, we should keep the migration scripts as it was, with that said, you are doing the right thing. -- 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]
