forked from tangled.org/core
Monorepo for Tangled — https://tangled.org

appview/db: rework issue comments

more constraints and a new "reply_to" column for threading.

Signed-off-by: oppiliappan <me@oppi.li>

oppi.li 11713b90 cd2126cf

verified
Changed files
+5 -11
appview
+5 -11
appview/ingester.go
··· 885 885 record := tangled.RepoIssueComment{} 886 886 err = json.Unmarshal(raw, &record) 887 887 if err != nil { 888 - l.Error("invalid record", "err", err) 889 - return err 888 + return fmt.Errorf("invalid record: %w", err) 890 889 } 891 890 892 891 comment, err := db.IssueCommentFromRecord(ddb, did, rkey, record) 893 892 if err != nil { 894 - l.Error("failed to parse comment from record", "err", err) 895 - return err 893 + return fmt.Errorf("failed to parse comment from record: %w", err) 896 894 } 897 895 898 896 sanitizer := markup.NewSanitizer() ··· 902 900 903 901 err = db.NewIssueComment(ddb, &comment) 904 902 if err != nil { 905 - l.Error("failed to create issue comment", "err", err) 906 - return err 903 + return fmt.Errorf("failed to create issue comment: %w", err) 907 904 } 908 905 909 906 return nil ··· 913 910 record := tangled.RepoIssueComment{} 914 911 err = json.Unmarshal(raw, &record) 915 912 if err != nil { 916 - l.Error("invalid record", "err", err) 917 - return err 913 + return fmt.Errorf("invalid record: %w", err) 918 914 } 919 915 920 916 sanitizer := markup.NewSanitizer() ··· 924 920 925 921 err = db.UpdateCommentByRkey(ddb, did, rkey, record.Body) 926 922 if err != nil { 927 - l.Error("failed to update issue comment", "err", err) 928 - return err 923 + return fmt.Errorf("failed to update issue comment: %w", err) 929 924 } 930 925 931 926 return nil 932 927 933 928 case models.CommitOperationDelete: 934 929 if err := db.DeleteCommentByRkey(ddb, did, rkey); err != nil { 935 - l.Error("failed to delete", "err", err) 936 930 return fmt.Errorf("failed to delete issue comment record: %w", err) 937 931 } 938 932