api/paidAction/itemCreate.js

@@ -227,7 +227,7 @@ export async function onPaid ({ invoice, id }, context) {
   for (const { refereeItem } of item.itemReferrers) {
     notifyItemMention({ models, referrerItem: item, refereeItem }).catch(console.error)
   }
-  notifyUserSubscribers({ models: tx, item }).catch(console.error)
+  notifyUserSubscribers({ models, item }).catch(console.error)
   notifyTerritorySubscribers({ models, item }).catch(console.error)
 }
 

components/text.js

@@ -22,7 +22,6 @@ import { UNKNOWN_LINK_REL } from '@/lib/constants'
 import isEqual from 'lodash/isEqual'
 import UserPopover from './user-popover'
 import ItemPopover from './item-popover'
-import classNames from 'classnames'
 
 // Explicitely defined start/end tags & which CSS class from text.module.css to apply
 export const rehypeSuperscript = () => rehypeStyler('<sup>', '</sup>', styles.superscript)
@@ -265,7 +264,7 @@ export default memo(function Text ({ rel, imgproxyUrls, children, tab, itemId, o
   const rehypePlugins = useMemo(() => [rehypeInlineCodeProperty, rehypeSuperscript, rehypeSubscript], [])
 
   return (
-    <div className={classNames(styles.text, topLevel && styles.topLevel, show ? styles.textUncontained : overflowing && styles.textContained)} ref={containerRef}>
+    <div className={`${styles.text} ${show ? styles.textUncontained : overflowing ? styles.textContained : ''}`} ref={containerRef}>
       <ReactMarkdown
         components={components}
         remarkPlugins={remarkPlugins}

components/text.module.css

@@ -106,12 +106,6 @@
     margin-bottom: .5rem;
     white-space: pre-wrap;
     word-break: break-word;
-    --grid-gap: 0.5rem;
-}
-
-.text.topLevel .p {
-    margin-bottom: 0.75rem;
-    --grid-gap: 0.75rem;
 }
 
 .text pre {
@@ -134,55 +128,17 @@
     margin-bottom: .5rem;
 }
 
-.mediaContainer {
+.text .mediaContainer {
     display: block;
+    margin-top: .5rem;
     margin-bottom: .5rem;
-    width: calc(100% - var(--grid-gap));
-    max-width: calc(100% - var(--grid-gap));
+    width: 100%;
     height: auto;
     overflow: hidden;
     max-height: 25vh;
     aspect-ratio: var(--aspect-ratio);
 }
 
-.p:has(> .mediaContainer:only-child) ~ .p:has(> .mediaContainer:only-child) {
-    display: inline-block;
-    width: min-content;
-    max-width: calc(100% - var(--grid-gap));
-    margin-bottom: 0;
-}
-
-.mediaContainer ~ .mediaContainer {
-    display: inline-block;
-    width: min-content;
-    margin-right: var(--grid-gap);
-}
-
-.p:has(> .mediaContainer:only-child) ~ .p:has(> .mediaContainer:only-child) > .mediaContainer:only-child {
-    margin-right: var(--grid-gap);
-}
-
-.p:has(> .mediaContainer:only-child):has(+ .p > .mediaContainer:only-child) {
-    display: inline-block;
-    width: min-content;
-    max-width: calc(100% - var(--grid-gap));
-    margin-bottom: 0;
-}
-
-.p:has(> .mediaContainer:only-child):has(+ .p > .mediaContainer:only-child) > .mediaContainer:only-child {
-    margin-right: var(--grid-gap);
-}
-
-.mediaContainer:has(+ .mediaContainer) {
-    display: inline-block;
-    width: min-content;
-    margin-right: var(--grid-gap);
-}
-
-.mediaContainer:first-child:has(+ .mediaContainer) {
-    margin-top: var(--grid-gap);
-}
-
 .mediaContainer img, .mediaContainer video {
     display: block;
     object-fit: contain;
@@ -190,7 +146,6 @@
     max-height: inherit;
     height: 100%;
     aspect-ratio: var(--aspect-ratio);
-    block-size: stretch;
 }
 
 .mediaContainer img {
@@ -200,6 +155,7 @@
 }
 
 .mediaContainer.topLevel {
+    margin-top: .75rem;
     margin-bottom: .75rem;
     max-height: 35vh;
 }
@@ -234,7 +190,7 @@ img.fullScreen {
     margin-bottom: 0;
 }
 
-.text li {
+.text li+li {
     margin-top: .25rem;
 }
 
@@ -304,12 +260,12 @@ img.fullScreen {
 
 .videoWrapper {
     max-width: 320px;
-    margin: 0.5rem 0;
+    padding-right: 15px;
+    margin: '0.5rem 0',
 }
 
 .videoWrapper.topLevel {
     max-width: 640px;
-    margin: 0.75rem 0;
 }
 
 .videoContainer {

lib/webPush.js

@@ -129,15 +129,7 @@ export const notifyUserSubscribers = async ({ models, item }) => {
     INNER JOIN users ON users.id = "UserSubscription"."followeeId"
     WHERE "followeeId" = $1 AND ${isPost ? '"postsSubscribedAt"' : '"commentsSubscribedAt"'} IS NOT NULL
     AND NOT EXISTS (SELECT 1 FROM "Mute" WHERE "Mute"."muterId" = "UserSubscription"."followerId" AND "Mute"."mutedId" = $1)
-    -- ignore subscription if user was already notified of item as a reply
-    AND NOT EXISTS (
-      SELECT 1 FROM "Reply"
-      INNER JOIN users follower ON follower.id = "UserSubscription"."followerId"
-      WHERE "Reply"."itemId" = $2
-      AND "Reply"."ancestorUserId" = follower.id
-      AND follower."noteAllDescendants"
-    )
-    `, Number(item.userId), Number(item.id))
+    `, Number(item.userId))
     const subType = isPost ? 'POST' : 'COMMENT'
     const tag = `FOLLOW-${item.userId}-${subType}`
     await Promise.allSettled(userSubsExcludingMutes.map(({ followerId, followeeName }) => sendUserNotification(followerId, {

pages/settings/index.js

@@ -107,8 +107,8 @@ export default function Settings ({ ssrData }) {
   return (
     <Layout>
       <div className='pb-3 w-100 mt-2' style={{ maxWidth: '600px' }}>
-        <SettingsHeader />
         {hasOnlyOneAuthMethod(settings?.authMethods) && <AuthBanner />}
+        <SettingsHeader />
         <Form
           initial={{
             tipDefault: settings?.tipDefault || 21,

styles/item.module.css

@@ -17,7 +17,6 @@
 
 .fullItemContainer {
     margin-bottom: .5rem;
-    padding-right: 15px;
 }
 
 .fullItemContainer:empty {

wallets/index.js

@@ -45,6 +45,12 @@ export function useWallet (name) {
     logger.info('payments disabled')
   }, [name, me, logger])
 
+  if (wallet) {
+    wallet.isConfigured = _isConfigured
+    wallet.enablePayments = enablePayments
+    wallet.disablePayments = disablePayments
+  }
+
   const status = config?.enabled ? Status.Enabled : Status.Initialized
   const enabled = status === Status.Enabled
   const priority = config?.priority
@@ -89,30 +95,21 @@ export function useWallet (name) {
 
   if (!wallet) return null
 
-  // Assign everything to wallet object so every function that is passed this wallet object in this
-  // `useWallet` hook has access to all others via the reference to it.
-  // Essentially, you can now use functions like `enablePayments` _inside_ of functions that are
-  // called by `useWallet` even before enablePayments is defined and not only in functions
-  // that use the return value of `useWallet`.
-  wallet.isConfigured = _isConfigured
-  wallet.enablePayments = enablePayments
-  wallet.disablePayments = disablePayments
-  wallet.canSend = !!wallet.sendPayment
-  wallet.canReceive = !!wallet.createInvoice
-  wallet.config = config
-  wallet.save = save
-  wallet.delete = delete_
-  wallet.deleteLogs = deleteLogs
-  wallet.setPriority = setPriority
-  wallet.hasConfig = hasConfig
-  wallet.status = status
-  wallet.enabled = enabled
-  wallet.priority = priority
-  wallet.logger = logger
-
-  // can't assign sendPayment to wallet object because it already exists
-  // as an imported function and thus can't be overwritten
-  return { ...wallet, sendPayment }
+  return {
+    ...wallet,
+    canSend: !!wallet.sendPayment,
+    sendPayment,
+    config,
+    save,
+    delete: delete_,
+    deleteLogs,
+    setPriority,
+    hasConfig,
+    status,
+    enabled,
+    priority,
+    logger
+  }
 }
 
 function extractConfig (fields, config, client) {