Feature #19885
closedRaise time entries comments limit to 1024
0%
Description
See: source:/trunk/app/models/time_entry.rb#L42
The comments
field in the time_entries
table is defined as having a length of 1024 characters:
Click to expand @time_entries@ table definition...
Yet, user comments are truncated at 255:
Let's not do that. :)
Note: This patch (partly) makes the comment field mandatory. That might not be what you want. It would be nice to have a way to require this field without patching the application though. :)
Related issues
Updated by Future Link Corporation pkgsrc team about 9 years ago
- Copied to Feature #19886: Raise wiki edits comments limit to 1024 added
Updated by Future Link Corporation pkgsrc team about 9 years ago
Here is a patch for source:/trunk/app/views/timelog/_form.html.erb as well:
Again, noting that we're making the field mandatory, which might not be what you want (POLA).
Updated by Toshi MARUYAMA about 9 years ago
- Status changed from New to Needs feedback
"comments" of "time_entries" is defined as 255.
source:tags/3.0.3/db/migrate/032_create_time_entries.rb#L8
I think you changed manually.
Updated by Future Link Corporation pkgsrc team about 9 years ago
Yes, you are correct. We did. Can that field be made longer by default? The length of 255 characters is rather short and doesn't allow for detailed time entries.
Updated by Future Link Corporation pkgsrc team about 9 years ago
The diff you referenced doesn't apply to the time entries comment
field.
Updated by Toshi MARUYAMA about 9 years ago
Updated by Future Link Corporation pkgsrc team about 9 years ago
Updated by Toshi MARUYAMA about 9 years ago
- Tracker changed from Patch to Feature
- Status changed from Needs feedback to New
Because there is no completed patch, I change tracker.
Updated by Jean-Philippe Lang almost 9 years ago
- Subject changed from Comment field length for time entries is unnecessarily short to Raise time entries comments limit to 1024
- Status changed from New to Closed
- Assignee set to Jean-Philippe Lang
- Resolution set to Fixed
Limit raised to 1024 in r14621.