tangled.org
Signed-off-by: Seongmin Lee git@boltless.me
appview/db/comments.go
appview/db/comments.go
This file has not been changed.
appview/db/db.go
appview/db/db.go
This file has not been changed.
appview/db/pulls.go
appview/db/pulls.go
This patch was likely rebased, as context lines do not match.
appview/db/reference.go
appview/db/reference.go
This file has not been changed.
appview/models/comment.go
appview/models/comment.go
This file has not been changed.
appview/models/pull.go
appview/models/pull.go
This patch was likely rebased, as context lines do not match.
appview/notify/db/db.go
appview/notify/db/db.go
This file has not been changed.
appview/notify/merged_notifier.go
appview/notify/merged_notifier.go
This file has not been changed.
appview/notify/notifier.go
appview/notify/notifier.go
This file has not been changed.
appview/notify/posthog/notifier.go
appview/notify/posthog/notifier.go
This file has not been changed.
+7
-7
appview/pages/templates/repo/pulls/pull.html
+7
-7
appview/pages/templates/repo/pulls/pull.html
···
165
166
<div class="md:pl-[3.5rem] flex flex-col gap-2 mt-2 relative">
167
{{ range $cidx, $c := .Comments }}
168
-
<div id="comment-{{$c.Id}}" class="bg-white dark:bg-gray-800 rounded drop-shadow-sm py-2 px-4 relative w-full">
169
{{ if gt $cidx 0 }}
170
<div class="absolute left-8 -top-2 w-px h-2 bg-gray-300 dark:bg-gray-600"></div>
171
{{ end }}
172
<div class="text-sm text-gray-500 dark:text-gray-400 flex items-center gap-1">
173
-
{{ template "user/fragments/picHandleLink" $c.Did.String }}
174
<span class="before:content-['·']"></span>
175
-
<a class="text-gray-500 dark:text-gray-400 hover:text-gray-500 dark:hover:text-gray-300" href="#comment-{{.Id}}">{{ template "repo/fragments/time" $c.Created }}</a>
176
</div>
177
<div class="prose dark:prose-invert">
178
{{ $c.Body | markdown }}
···
505
{{ end }}
506
507
{{ define "submissionComment" }}
508
-
<div id="comment-{{.ID}}" class="flex gap-2 -ml-4 py-4 w-full mx-auto">
509
<!-- left column: profile picture -->
510
<div class="flex-shrink-0">
511
<img
512
-
src="{{ tinyAvatar .OwnerDid }}"
513
alt=""
514
class="rounded-full size-8 mr-1 border-2 border-gray-100 dark:border-gray-900"
515
/>
···
518
<div class="flex-1 min-w-0">
519
<!-- Row 1: Author and timestamp -->
520
<div class="text-sm text-gray-500 dark:text-gray-400 flex items-center gap-1">
521
-
<span>{{ resolve .OwnerDid }}</span>
522
<span class="before:content-['·']"></span>
523
-
<a class="text-gray-500 dark:text-gray-400 hover:text-gray-500 dark:hover:text-gray-300" href="#comment-{{.ID}}">
524
{{ template "repo/fragments/time" .Created }}
525
</a>
526
</div>
···
165
166
<div class="md:pl-[3.5rem] flex flex-col gap-2 mt-2 relative">
167
{{ range $cidx, $c := .Comments }}
168
+
<div id="comment-{{$c.ID}}" class="bg-white dark:bg-gray-800 rounded drop-shadow-sm py-2 px-4 relative w-full">
169
{{ if gt $cidx 0 }}
170
<div class="absolute left-8 -top-2 w-px h-2 bg-gray-300 dark:bg-gray-600"></div>
171
{{ end }}
172
<div class="text-sm text-gray-500 dark:text-gray-400 flex items-center gap-1">
173
+
{{ template "user/fragments/picHandleLink" $c.OwnerDid }}
174
<span class="before:content-['·']"></span>
175
+
<a class="text-gray-500 dark:text-gray-400 hover:text-gray-500 dark:hover:text-gray-300" href="#comment-{{.ID}}">{{ template "repo/fragments/time" $c.Created }}</a>
176
</div>
177
<div class="prose dark:prose-invert">
178
{{ $c.Body | markdown }}
···
505
{{ end }}
506
507
{{ define "submissionComment" }}
508
+
<div id="comment-{{.Id}}" class="flex gap-2 -ml-4 py-4 w-full mx-auto">
509
<!-- left column: profile picture -->
510
<div class="flex-shrink-0">
511
<img
512
+
src="{{ tinyAvatar .Did.String }}"
513
alt=""
514
class="rounded-full size-8 mr-1 border-2 border-gray-100 dark:border-gray-900"
515
/>
···
518
<div class="flex-1 min-w-0">
519
<!-- Row 1: Author and timestamp -->
520
<div class="text-sm text-gray-500 dark:text-gray-400 flex items-center gap-1">
521
+
<span>{{ resolve .Did.String }}</span>
522
<span class="before:content-['·']"></span>
523
+
<a class="text-gray-500 dark:text-gray-400 hover:text-gray-500 dark:hover:text-gray-300" href="#comment-{{.Id}}">
524
{{ template "repo/fragments/time" .Created }}
525
</a>
526
</div>
appview/pulls/opengraph.go
appview/pulls/opengraph.go
This file has not been changed.
appview/pulls/pulls.go
appview/pulls/pulls.go
This patch was likely rebased, as context lines do not match.
History
8 rounds
4 comments
boltless.me
submitted
#7
1 commit
expand
collapse
appview: replace
PullComment to Comment
Including db migration to migrate `issue_comments` and `pull_comments`
to unified `comments` table.
Signed-off-by: Seongmin Lee <git@boltless.me>
2/3 timeout, 1/3 success
expand
collapse
merge conflicts detected
expand
collapse
expand
collapse
- appview/notify/db/db.go:260
- appview/notify/merged_notifier.go:81
- appview/notify/notifier.go:22
- appview/pulls/opengraph.go:277
expand 0 comments
boltless.me
submitted
#6
1 commit
expand
collapse
appview: replace
PullComment to Comment
Including db migration to migrate `issue_comments` and `pull_comments`
to unified `comments` table.
Signed-off-by: Seongmin Lee <git@boltless.me>
2/3 failed, 1/3 success
expand
collapse
expand 0 comments
boltless.me
submitted
#5
1 commit
expand
collapse
appview: replace
PullComment to Comment
Including db migration to migrate `issue_comments` and `pull_comments`
to unified `comments` table.
Signed-off-by: Seongmin Lee <git@boltless.me>
3/3 success
expand
collapse
expand 0 comments
boltless.me
submitted
#4
1 commit
expand
collapse
appview: replace
PullComment to Comment
Including db migration to migrate `issue_comments` and `pull_comments`
to unified `comments` table.
Signed-off-by: Seongmin Lee <git@boltless.me>
3/3 success
expand
collapse
expand 1 comment
boltless.me
submitted
#3
1 commit
expand
collapse
appview: replace
PullComment to Comment
Including db migration to migrate `issue_comments` and `pull_comments`
to unified `comments` table.
Signed-off-by: Seongmin Lee <git@boltless.me>
1/3 failed, 2/3 success
expand
collapse
expand 0 comments
boltless.me
submitted
#2
1 commit
expand
collapse
appview: replace
PullComment to Comment
Including db migration to migrate `issue_comments` and `pull_comments`
to unified `comments` table.
Signed-off-by: Seongmin Lee <git@boltless.me>
1/3 failed, 2/3 success
expand
collapse
expand 0 comments
boltless.me
submitted
#1
1 commit
expand
collapse
appview: replace
PullComment to Comment
Signed-off-by: Seongmin Lee <git@boltless.me>
3/3 success
expand
collapse
expand 3 comments
I should've been more precise about my expression. Existing comments won't be dropped as they are stored in issue_comments and pull_comments table (I'm just dropping legacy table with same name there), but yeah this PR itself doesn't include the backward compatibility layer yet so old comments won't be rendered.
Honestly this PR hasn't been updated since you two suggested to just ingest old records. I'll push more commits on top of this stack.
As a side note: I'd suggest to not publish the new sh.tangled.comment lexicon yet.
This unified comment record can be used for PR review comments and it might make us to modify the schema again. I think it would be safest to publish this when PR redesign discussion is finished.
boltless.me
submitted
#0
1 commit
expand
collapse
appview: replace
PullComment to Comment
Signed-off-by: Seongmin Lee <git@boltless.me>
here, the sort order is reversed, it should be
.Before.