Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
To allow other modules to use messages related to tags (annotations) we must add a default method to VersioncontrolTag, ready to be overwritten by backends.
Comment | File | Size | Author |
---|---|---|---|
#1 | 1322648-api-level-tag-messages.patch | 520 bytes | Niklas Fiekas |
Comments
Comment #1
Niklas Fiekas CreditAttribution: Niklas Fiekas commentedFirst try.
Comment #2
Niklas Fiekas CreditAttribution: Niklas Fiekas commentedComment #3
sdboyer CreditAttribution: sdboyer commentedyep, just what i was looking for. commushing. now we can sort out what the actual implementation should look like in vc_git land.
Comment #4
Niklas Fiekas CreditAttribution: Niklas Fiekas commented:)
Comment #6
marvil07 CreditAttribution: marvil07 commentedI am really late here, but I would say we should follow the same pattern we use in operations. That means adding a data member(this sounds like we really need to review code) for the message, and a column to the versioncontrol_labels table, to actually store it, so we can access it from places like views(IMHO we do not want to call repositories for it).
Comment #7
sdboyer CreditAttribution: sdboyer commentedi don't want a data member. i want separate, backend-specific tables that can be joined in.
the data member is ok on operations b/c it's pretty well established across vcs types that you'll have a commit message. tags are not nearly so standardized, so i'd rather not put something in the generic table which is going to be used different ways, if at all, by different backends.
Comment #8
marvil07 CreditAttribution: marvil07 commentedMake sense. Opening #1666622: Support tag messages and re-closing this.