Skip to content

edit and delete actions for thread messages #505

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 14 commits into
base: staging
Choose a base branch
from
Open

edit and delete actions for thread messages #505

wants to merge 14 commits into from

Conversation

@sancheslfl
Copy link
Contributor

@sancheslfl sancheslfl commented Nov 9, 2025

Thread messages can now be edited and deleted

Closes #472

sancheslfl and others added 12 commits October 21, 2025 20:04
@sancheslfl
edit personal info and select new profile picture with hovering effect
6b48860 
Update .gitignore

git ignore update

stop tracking .vendor-new files
@sancheslfl
edit and upload profile picture
6658ba9
@sancheslfl
using Image component instead of native html source
3aec774
@Francisca105
fix: set cdnBaseURL to endpoint if custom domain is empty
9e74631
@Francisca105
Merge branch 'staging' into sancheslfl/435-edit-own-profile-picture
1402a6e
@Francisca105
feat: enhance profile image handling and upload functionality
9214b0e
@Francisca105
style: format code
e070413
@sancheslfl
fix(settings): correct Member.img access and add preview URL cleanup …
e96d12c 
…helper
@sancheslfl
test: test commit for husky
fe0ba8b
@sancheslfl
feat: add edit functionality with api integration
76194ab
@sancheslfl
fix: buttons with more visible appearance and edit button visible whi…
6c98957 
…le on edit mode
@sancheslfl
fix: edit and delete action work with optimistic thread creation
5b923e6
@sancheslfl sancheslfl linked an issue Nov 9, 2025 that may be closed by this pull request
Complete communications #472
Open
@Francisca105 Francisca105 changed the base branch from master to staging November 9, 2025 20:18
@sancheslfl sancheslfl marked this pull request as ready for review November 10, 2025 12:31
@sancheslfl
Copy link
Contributor Author

sancheslfl commented Nov 10, 2025

Note that in mutations/threads.ts file, there is a newly created type, as shown below, that is used in mutations/post.ts too. Check if that's the right place to put it.

export type EntityType = "speaker" | "company";

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@luckspt luckspt Awaiting requested review from luckspt

@Francisca105 Francisca105 Awaiting requested review from Francisca105

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Complete communications

3 participants

@sancheslfl @Francisca105