core/appview/state/ at 97ca49ba9114b18ca6ed024c1f4fb2ac63d006c3 · alice.pds.demo.boltless.dev/core · Tangled

alice.pds.demo.boltless.dev / core

0

This repository has no description

0

core / appview / state

1 folder 17 files

at 97ca49ba9114b18ca6ed024c1f4fb2ac63d006c3

eti appview: deduplicate newsletter widget, single-render timeline grid 3mo ago

appview: remove `@` from URLs and interface old URLs that refer to users with the `@` are redirected to the version without `@`. the leading motivation for this change is that valid atproto handles do not contain the prefix. it is purely stylistic. Signed-off-by: oppiliappan <me@oppi.li>

10 months ago

appview: deduplicate newsletter widget, single-render timeline grid the newsletter widget was being rendered twice on the timeline (once for mobile, once for desktop) via hidden/visible wrapper divs. both copies shared the same element ids (newsletter-widget, newsletter-widget-msg), so desktop submissions swapped into the hidden mobile copy and appeared to do nothing. collapse the two layouts into one responsive grid with tailwind order/col-start utilities so the widget lives in the dom exactly once, and drop the mutationobserver wiring in favour of a small delegated click listener that also handles gfi banner widening on dismiss. consolidate the three signup form variants (banner, widget, home hero) into a single newsletterForm fragment that takes a dict(Id, Variant). each instance gets a unique response-span id (newsletter-msg-<Id>) so multiple forms can coexist on a page without id collisions. move the handler's inline html/tailwind response strings into a newsletterResponse template rendered through pages, so the response id round-trips with the form's hx-target via an hx-vals target field. drop the stray blank line in base.html. Signed-off-by: eti <eti@eti.tf>

3 months ago

appview: remove `oauth.User` type This is not required anymore. `MultiAccountUser` can just hold active user DID. Signed-off-by: Seongmin Lee <git@boltless.me>

6 months ago

appview/state: replace log package with structured slog logging Replace all log.Println/log.Printf calls across follow.go, star.go, reaction.go, gfi.go, and profile.go with s.logger.Error/Warn/Info using structured key-value pairs. Each handler opens with a child logger via s.logger.With("handler", "FuncName"). Firehose-idempotent delete failures (follow, star, reaction) use Warn; all other failures use Error. Signed-off-by: Matías Insaurralde <matias@insaurral.de>

4 months ago

appview/notify: add Clone notifier, enqueue posthog event for clones Signed-off-by: Anirudh Oppiliappan <anirudh@tangled.org>

4 months ago

appview/sites: use repoIdentifier for archive requests we could just use the knotmirror here directly instead of deploying from the knot, but we'd need the knotmirror to update before the site. for now just deploy from the knot. Signed-off-by: oppiliappan <me@oppi.li>

4 months ago

appview,knotserver: pref-handle display + identity ingest Lewis: May this revision serve well! <lewis@tangled.org>

4 months ago

appview/state: fix open redirect via return_url after OAuth login Validate return_url before storing it in the session: only relative paths starting with "/" (and not "//") are accepted. Anything else — absolute URLs and protocol-relative URLs — is replaced with "/". Add tests covering the accepted and rejected cases. Signed-off-by: Matías Insaurralde <matias@insaurral.de>

4 months ago

appview/state: refactor manifest.json incorporates changes suggested by @boltless.me Co-authored-by: Seongmin Lee <git@boltless.me> Signed-off-by: oppiliappan <me@oppi.li>

7 months ago

appview,knotserver: pref-handle display + identity ingest Lewis: May this revision serve well! <lewis@tangled.org>

4 months ago

appview: remove `oauth.User` type This is not required anymore. `MultiAccountUser` can just hold active user DID. Signed-off-by: Seongmin Lee <git@boltless.me>

6 months ago

appview: upgrade to resend-go/v3, add newsletter signup upgrade to resend-go/v3 and add a newsletter signup endpoint. replace the join CTA with a newsletter signup form. rework the timeline page to be a two-column layout on desktop. Signed-off-by: Anirudh Oppiliappan <anirudh@tangled.org> Signed-off-by: eti <eti@eti.tf>

3 months ago

appview/state: add posthog capture for search Sets the distinct id to 'anonymous' for logged out searches. Let's stick to this for any future logged out events. Signed-off-by: Anirudh Oppiliappan <anirudh@tangled.org>

3 months ago

appview/state: add test for parseSortParam Signed-off-by: Matías Insaurralde <matias@insaurral.de>

4 months ago

spindlestream.go

appview/db: propagate context through GetSpindles and AddPipelineStatus Update all call sites of db.GetSpindles and db.AddPipelineStatus to pass a context.Context, enabling proper cancellation and timeout propagation to the database layer. Signed-off-by: Matías Insaurralde <matias@insaurral.de>

4 months ago

*: fmt Signed-off-by: Anirudh Oppiliappan <anirudh@tangled.org>

3 months ago

appview: deduplicate newsletter widget, single-render timeline grid the newsletter widget was being rendered twice on the timeline (once for mobile, once for desktop) via hidden/visible wrapper divs. both copies shared the same element ids (newsletter-widget, newsletter-widget-msg), so desktop submissions swapped into the hidden mobile copy and appeared to do nothing. collapse the two layouts into one responsive grid with tailwind order/col-start utilities so the widget lives in the dom exactly once, and drop the mutationobserver wiring in favour of a small delegated click listener that also handles gfi banner widening on dismiss. consolidate the three signup form variants (banner, widget, home hero) into a single newsletterForm fragment that takes a dict(Id, Variant). each instance gets a unique response-span id (newsletter-msg-<Id>) so multiple forms can coexist on a page without id collisions. move the handler's inline html/tailwind response strings into a newsletterResponse template rendered through pages, so the response id round-trips with the form's hx-target via an hx-vals target field. drop the stray blank line in base.html. Signed-off-by: eti <eti@eti.tf>

3 months ago

appview: remove `oauth.User` type This is not required anymore. `MultiAccountUser` can just hold active user DID. Signed-off-by: Seongmin Lee <git@boltless.me>

6 months ago