Goldloli opened a new pull request, #9303:
URL: https://github.com/apache/devlake/pull/9303
Fixes #9284
## Description
Adding a Tempo team whose summary exceeds 255 characters to a connection's
data scope fails with `ERROR: value too long for type character varying(255)
(SQLSTATE 22001)`, because the `TempoTeam.Summary` column is declared as
`varchar(255)` while Tempo allows free-text summaries of any length.
Changes:
- `models/team.go`: `Summary` is now `gorm:"type:text"`
- `models/migrationscripts/20261102_change_team_summary_type.go`: new
migration calling `ModifyColumnType("_tool_tempo_teams", "summary", "text")`
(the same approach the bitbucket plugin used for its `component` column),
registered in `All()`
## Verification
- `go build ./plugins/tempo/...` and `go vet ./plugins/tempo/...` — clean
- `go test ./plugins/tempo/tasks/` — ok (unit tests pass)
- `plugins/tempo/e2e` requires a live `E2E_DB_URL` database and fails
identically on a clean checkout without one, so the migration path itself is
covered by CI
- gofmt clean; no new models, so `GetTablesInfo()` is unchanged
--
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]
