dosubot[bot] commented on issue #8835: URL: https://github.com/apache/incubator-devlake/issues/8835#issuecomment-4244036470
<!-- Greeting --> Hi @JAORMX! I'm [Dosu](https://go.dosu.dev/dosubot) and Iām helping the incubator-devlake team. <!-- Answer --> Your analysis and suggested fix are spot-on ā this is the same pattern that was fixed for the gh-copilot plugin in [#8779](https://github.com/apache/incubator-devlake/pull/8779), where explicit MySQL-specific GORM type tags (`gorm:"type:datetime"`) were removed to let GORM handle type mapping automatically per database backend [[1]](https://github.com/apache/incubator-devlake/issues/8778). One note: I wasn't able to find the Asana plugin or the file `backend/plugins/asana/models/migrationscripts/20250212_add_task_transformation_fields.go` in the current repository. It's possible this was added in v1.0.3-beta10 but hasn't been merged to the main branch yet, or lives in a different branch. Could you confirm which branch or release artifact you're seeing this in? In any case, the fix approach ā removing `gorm:"type:double"` and letting GORM infer the column type from `*float64` ā is exactly what was done in #8779 and is the recommended pattern in DevLake to stay database-agnostic. <!-- Dosu Comment Footer --> *To reply, just mention [@dosu](https://go.dosu.dev/dosubot?utm_source=github&utm_medium=bot-comment&utm_campaign=github-assistant&utm_content=brand-link).* --- <sup>How did I do? [Good](https://app.dosu.dev/response-feedback?utm_source=github&utm_medium=bot-comment&utm_campaign=github-assistant&utm_content=feedback-great_response#message_id=81742d09-b169-46f9-a2c4-7516ae540261&feedback_type=great_response) | [Irrelevant](https://app.dosu.dev/response-feedback?utm_source=github&utm_medium=bot-comment&utm_campaign=github-assistant&utm_content=feedback-irrelevant_answer#message_id=81742d09-b169-46f9-a2c4-7516ae540261&feedback_type=irrelevant_answer) | [Incorrect](https://app.dosu.dev/response-feedback?utm_source=github&utm_medium=bot-comment&utm_campaign=github-assistant&utm_content=feedback-incorrect_sources#message_id=81742d09-b169-46f9-a2c4-7516ae540261&feedback_type=incorrect_sources) | [Verbose](https://app.dosu.dev/response-feedback?utm_source=github&utm_medium=bot-comment&utm_campaign=github-assistant&utm_content=feedback-too_verbose#message_id=81742d09-b169-46f9-a2c4-7516ae540261&feedback_type=too_verbose) | [Hallucination](https:// app.dosu.dev/response-feedback?utm_source=github&utm_medium=bot-comment&utm_campaign=github-assistant&utm_content=feedback-hallucination#message_id=81742d09-b169-46f9-a2c4-7516ae540261&feedback_type=hallucination) | [Report š](https://app.dosu.dev/response-feedback?utm_source=github&utm_medium=bot-comment&utm_campaign=github-assistant&utm_content=feedback-bug_report#message_id=81742d09-b169-46f9-a2c4-7516ae540261&feedback_type=bug_report) | [Other](https://app.dosu.dev/response-feedback?utm_source=github&utm_medium=bot-comment&utm_campaign=github-assistant&utm_content=feedback-other#message_id=81742d09-b169-46f9-a2c4-7516ae540261&feedback_type=other)</sup> [](https://app.dosu.dev/b4e8e847-d479-4541-83a8-d88d83fea5c9/ask?utm_source=github&utm_medium=bot-comment&utm_campaign=github-assistant&utm_content=chat-badge) [](https://cursor.com/link/prompt?text=Your%20analysis%20and%20suggested%20fix%20are%20spot-on%20%E2%80%94%20this%20is%20the%20same%20pattern%20that%20was%20fixed%20for%20the%20gh-copilot%20plugin%20in%20%5B%238779%5D%28https%3A//github.com/apache/incubator-devlake/pull/8779%29%2C%20where%20explicit%20MySQL-specific%20GORM%20type%20tags%20%28%60gorm%3A%22type%3Adatetime%22%60%29%20were%20removed%20to%20let%20GORM%20handle%20type%20mapping%20automatically%20per%20database%20backend%20%5B%5B1%5D%5D%28https%3A//github.com/apache/incubator-devlake/issues/8778%29.%0A%0AOne%20note%3A%20I%20wasn%27t%20able%20to%20find%20the%20Asana%20plugin%20or%20the%20file%20%60backend/plugins/asana/models/migrationscripts/20250212_a dd_task_transformation_fields.go%60%20in%20the%20current%20repository.%20It%27s%20possible%20this%20was%20added%20in%20v1.0.3-beta10%20but%20hasn%27t%20been%20merged%20to%20the%20main%20branch%20yet%2C%20or%20lives%20in%20a%20different%20branch.%20Could%20you%20confirm%20which%20branch%20or%20release%20artifact%20you%27re%20seeing%20this%20in%3F%0A%0AIn%20any%20case%2C%20the%20fix%20approach%20%E2%80%94%20removing%20%60gorm%3A%22type%3Adouble%22%60%20and%20letting%20GORM%20infer%20the%20column%20type%20from%20%60%2Afloat64%60%20%E2%80%94%20is%20exactly%20what%20was%20done%20in%20%238779%20and%20is%20the%20recommended%20pattern%20in%20DevLake%20to%20stay%20database-agnostic.) [](https://go.dosu.dev/discord-bot?utm_source=github&utm_medium=bot-comment&utm_campaign=github-assistant&utm_content=join-discord) [](https://twitter.co m/intent/tweet?text=%40dosu_ai%20helped%20me%20solve%20this%20issue!&url=https%3A//github.com/apache/incubator-devlake/issues/8835) -- 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]
