Remove post descriptions from timelines by default
This commit is contained in:
parent
9093f38b47
commit
1b97bbb2d9
|
@ -479,6 +479,7 @@ This is common for custom types like:
|
||||||
*/
|
*/
|
||||||
var queryableKinds = []reflect.Kind{
|
var queryableKinds = []reflect.Kind{
|
||||||
reflect.Int,
|
reflect.Int,
|
||||||
|
reflect.String,
|
||||||
}
|
}
|
||||||
|
|
||||||
/*
|
/*
|
||||||
|
|
|
@ -69,6 +69,8 @@ type TimelineQuery struct {
|
||||||
SkipSnippets bool
|
SkipSnippets bool
|
||||||
SkipPosts bool
|
SkipPosts bool
|
||||||
|
|
||||||
|
IncludePostDescription bool
|
||||||
|
|
||||||
Limit, Offset int
|
Limit, Offset int
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -252,6 +254,10 @@ func FetchTimeline(
|
||||||
if results[idx].Owner != nil {
|
if results[idx].Owner != nil {
|
||||||
results[idx].Owner.AvatarAsset = results[idx].AvatarAsset
|
results[idx].Owner.AvatarAsset = results[idx].AvatarAsset
|
||||||
}
|
}
|
||||||
|
if results[idx].Item.Type == models.TimelineItemTypePost && !q.IncludePostDescription {
|
||||||
|
results[idx].Item.RawDescription = ""
|
||||||
|
results[idx].Item.ParsedDescription = ""
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
var projectIds []int
|
var projectIds []int
|
||||||
|
|
|
@ -6,10 +6,12 @@ import (
|
||||||
"github.com/google/uuid"
|
"github.com/google/uuid"
|
||||||
)
|
)
|
||||||
|
|
||||||
|
type TimelineItemType string
|
||||||
|
|
||||||
const (
|
const (
|
||||||
TimelineItemTypeSnippet = "snippet"
|
TimelineItemTypeSnippet TimelineItemType = "snippet"
|
||||||
TimelineItemTypePost = "post"
|
TimelineItemTypePost TimelineItemType = "post"
|
||||||
TimelineItemTypeStream = "stream" // NOTE(asaf): Not currently supported
|
TimelineItemTypeStream TimelineItemType = "stream" // NOTE(asaf): Not currently supported
|
||||||
)
|
)
|
||||||
|
|
||||||
// NOTE(asaf): This is a virtual model made up of several different tables
|
// NOTE(asaf): This is a virtual model made up of several different tables
|
||||||
|
@ -18,7 +20,7 @@ type TimelineItem struct {
|
||||||
// NOTE(asaf): Several different items can have the same ID because we're merging several tables
|
// NOTE(asaf): Several different items can have the same ID because we're merging several tables
|
||||||
ID int `db:"id"`
|
ID int `db:"id"`
|
||||||
Date time.Time `db:"\"when\""`
|
Date time.Time `db:"\"when\""`
|
||||||
Type string `db:"timeline_type"`
|
Type TimelineItemType `db:"timeline_type"`
|
||||||
OwnerID int `db:"owner_id"`
|
OwnerID int `db:"owner_id"`
|
||||||
Title string `db:"title"`
|
Title string `db:"title"`
|
||||||
ParsedDescription string `db:"parsed_desc"`
|
ParsedDescription string `db:"parsed_desc"`
|
||||||
|
|
|
@ -56,7 +56,9 @@
|
||||||
- TikTok?
|
- TikTok?
|
||||||
- Trello?
|
- Trello?
|
||||||
- [ ] Handle empty avatar URLs correctly in various places (render as theme-dependent default)
|
- [ ] Handle empty avatar URLs correctly in various places (render as theme-dependent default)
|
||||||
- [ ] Convert to new color vars
|
- [x] Convert to new color vars
|
||||||
|
- [ ] Make snippet descriptions partially collapse by default
|
||||||
|
- [ ] Make the home page remember which tab you were on
|
||||||
- [ ] Resolve TODO(redesign) comments
|
- [ ] Resolve TODO(redesign) comments
|
||||||
|
|
||||||
stack!
|
stack!
|
||||||
|
|
Loading…
Reference in New Issue