ckolderup / postmarks

a single-user bookmarking website designed to live on the Fediverse
https://postmarks.glitch.me
MIT License
464 stars 38 forks source link

Add profile attachments (and a way of updating followers with new profile info) #190

Open TomCasavant opened 5 months ago

TomCasavant commented 5 months ago

Addresses #189

I didn't add any styles to the new profile fields, so here's what they look like right now: image

And then as of right now the only way I have to update current followers with this any new profile updates is to uncomment this line in server.js (EDIT: Moved updateProfile call to fix linting issues)

//updateProfile(account, domain)

image

I have it commented it out by default because I assume we don't want to send a message to every follower whenever the server boots up (and eventually I assume we'll have a way to update your profile from the admin page? In which case we'd just trigger updateProfile whenever that gets changed)

Maybe this belongs in a different issue/feature request but I was thinking since we already have a variable for the mastodon account we could probably include that as a profile field by default if it exists

Lastly, I just noticed this as I was making this PR, in this current setup it adds up profile updates in the total post count, that gets fixed if I modify this function to:

export async function getMessageCount() {
  return (await db?.get('select count(message) as count from messages where bookmark_id is not null'))?.count;
}

But I'd like clarification on if I'm storing these activities/updates in the right spot in the database or if there are any other non-bookmark activities that we are already storing