tangled.org / core
Monorepo for Tangled tangled.org
fork atom

appview: replace IssueComment to Comment #866

open
opened by boltless.me targeting master from sl/wnrvrwyvrlzo
STACK 4
#867 appview/notify: merge new comment events into one
3/3
0
#2
#866 appview: replace IssueComment to Comment
2/3
0
#2
#865 appview: replace PullComment to Comment
2/3
0
#2
#864 lexicons: add general sh.tangled.comment lexicon
3/3
0
#2
0
by boltless.me 0 comments
expand 1 commit
c828cadb
appview: replace IssueComment to Comment
build.yml
fmt.yml
test.yml
sign up or login to add to the discussion
1
by boltless.me 1 comment
expand 1 commit
9936098c
appview: replace IssueComment to Comment
oppi.li

imo: we should still continue to ingest sh.tangled.issue.comment (and pull comments), but just try to convert to the regular lexicon. this would only be used for backfill purposes, newer appviews should ideally not create the old NSID anymore.

the way i see it, the new lexicon is just an alias for the old ones, so the ingesters should be backwards compatible.

i will do a deeper dive into the code itself shortly.

build.yml
fmt.yml
test.yml
sign up or login to add to the discussion
2
by boltless.me 0 comments
expand 1 commit
4e6e6251
appview: replace IssueComment to Comment
build.yml
fmt.yml
test.yml
merge conflicts detected
sign up or login to add to the discussion
Labels

None yet.

assignee

None yet.

Participants 2
AT URI
at://did:plc:xasnlahkri4ewmbuzly2rlc5/sh.tangled.repo.pull/3m7iohv2yb622