Together App Minutes 04/29/25

[ShaperEvangelen]

4/29

General:
reviewed PR checklist in the wiki's contributing section
keep local and branch names as simple as possible
commits should reference the issue they're addressing
be careful about using octothorpes when naming things
title your PR with a description of the issue
if you get comments on your pull request, please take them to heart
Also (from DVKR), read along the active PRs and ask questions about it. That can be a huge learning opportunity!

rebase vs merge:
rebase is more organized
squash lets you rename the commits
rebase uses a linear history, will change the date of your history to when it was merged
merge strategy works in parallel and also gives a commit timestamp
sticking with rebase

Moderator Privileges:
this is the main feature we ant to focus on.
architecture design?
Boolean flag?
who's in charge of turning it on and off?
User with extra permissions or full admin privileges?
mvp currently is "what's currently preventing a user from deleting another user's work?"
databade admin is the "true admin"
how can we pass "isAdmin" to the front end?

Shan:
improved mobile model!
taking the lead on storybook

Emme:
lead on updating MongoDB, Express, and dependencies

StevenMoses:
lead on discord/GitHub names update pending

Ash:
H2 changed to H1 for the pull request

eric:
it used to be testing was optional, still staying nimble for now
if you're confident with testing go ahead, and if not, make a separate issue so others can help with testing

HTTP/HTTPS bug between Passport.JS and Discord:
Evie's looking into it.