Skip to content
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

✨ feat(web): Convo Viewer #41

Merged
merged 41 commits into from
Feb 2, 2024
Merged

✨ feat(web): Convo Viewer #41

merged 41 commits into from
Feb 2, 2024

Conversation

McPizza0
Copy link
Member

What does this PR do?

adds the initial convo list in the conversations sidebar
and conversation pages to load in convo messages
still needs some addition work for the following:

  • reply input and endpoints

  • convo Actions

  • New feature (non-breaking change which adds functionality)

Checklist

  • Read Contributing Guide
  • Self-reviewed my own code
  • Tested my code in a local environment
  • Commented on my code in hard-to-understand areas
  • Checked for warnings, there are none
  • Removed all console.logs
  • Merged the latest changes from main onto my branch with git pull origin main
  • My changes don't cause any responsiveness issues

Appreciated

  • If a UI change was made: Added a screen recording or screenshots to this PR

image

@McPizza0
fix user redirect link
7b132f9
@McPizza0
update avatar component
170a6e5
@McPizza0
new convo redirect
3f7953b
@McPizza0
fix navbar avatar bug
dbcee16
@McPizza0
update UnUiAvatar to accept correct colors
110bc2d
@McPizza0
DB update convo relationships to use "participants"
8fdc567
@McPizza0
dedicated convo avatar
c6497a3
@McPizza0
inital convo messages components
bd1e820
@McPizza0
update convo page
792beab
@McPizza0
newConvo page update types
09487c9
@McPizza0
trpc update convo + entries endpoints
c73e72e
@McPizza0
remove misc
5844718
@McPizza0
update tailwind safelist
d51bdd3
@McPizza0
Merge branch 'main' into feat/convoViewer
9ae3f2e
@McPizza0
Merge branch 'main' into feat/convoViewer
e664996
@McPizza0
Merge branch 'main' into feat/convoViewer
60ef701
@McPizza0
Merge branch 'main' into feat/convoViewer
0960be2
@McPizza0
fix(db): 🐛 set convo lastUpdatedAt to notNull
cb4d766
@McPizza0
feat(web): ✨ add getUserQuery trpc endpoint
1ebcd96
@McPizza0
remove avatar list
181e111
@McPizza0
add popover component
b2d03c2
@McPizza0
add virtualScroller component plugin
6222f76
@McPizza0
fix load avatarId
4cfc55c
@McPizza0
remove sidebar comment hydration error
a6215fe
@McPizza0
remove comment
14606a5
@McPizza0
switch convo loading to use status
1f1f438
@McPizza0
add query keys
9d25e13
@McPizza0
implement convolist
e6c1b4f
@McPizza0 McPizza0 requested a review from cstrnt January 30, 2024 20:27
@McPizza0
Copy link
Member Author

there is an upstream issue that blocks rendering of the conversation under specific circumstances
tracking here: nuxt/nuxt#25214 (comment) and here: vuejs/core#10184

cstrnt
cstrnt suggested changes Feb 1, 2024
View reviewed changes
apps/web-app/components/convos/convoAvatar.vue Outdated Show resolved Hide resolved
apps/web-app/components/convos/convoList.vue Outdated Show resolved Hide resolved
apps/web-app/components/convos/convoList.vue Outdated Show resolved Hide resolved
apps/web-app/components/convos/convoListItem.vue Outdated Show resolved Hide resolved
apps/web-app/components/convos/convoListItem.vue Outdated Show resolved Hide resolved
apps/web-app/components/convos/convoMessages.vue Outdated Show resolved Hide resolved
apps/web-app/components/convos/convoMessages.vue Outdated Show resolved Hide resolved
apps/web-app/components/convos/convoMessages.vue Outdated Show resolved Hide resolved
apps/web-app/components/convos/convoMessages.vue Outdated Show resolved Hide resolved
apps/web-app/components/un/ui-avatar-plus.vue Outdated Show resolved Hide resolved
McPizza0 and others added 11 commits February 1, 2024 13:17
@McPizza0 @cstrnt
Update apps/web-app/components/convos/convoList.vue
bf7b68a 
Co-authored-by: Tim Raderschad <[email protected]>
@McPizza0 @cstrnt
Update apps/web-app/components/convos/convoMessages.vue
dab24b7 
Co-authored-by: Tim Raderschad <[email protected]>
@McPizza0 @cstrnt
Update apps/web-app/components/convos/convoMessages.vue
c7ba605 
Co-authored-by: Tim Raderschad <[email protected]>
@McPizza0 @cstrnt
Update apps/web-app/components/un/ui-avatar-plus.vue
d2f772c 
Co-authored-by: Tim Raderschad <[email protected]>
@McPizza0 @cstrnt
Update apps/web-app/components/convos/convoMessages.vue
7937232 
Co-authored-by: Tim Raderschad <[email protected]>
@McPizza0
add new convo util useParticipantData
224f58b
@McPizza0
update convoAvatar component
f155b4f
@McPizza0
update utils to use types imports
1829d6f
@McPizza0
update convo view to use new util
7a2188d
@McPizza0
refactor convoListItem codes
8413d86
@McPizza0
update code/remove comments
83686cf
@McPizza0
Copy link
Member Author

McPizza0 commented Feb 1, 2024

changes made

@McPizza0 McPizza0 requested a review from cstrnt February 1, 2024 14:23
@McPizza0 McPizza0 merged commit 7504116 into main Feb 2, 2024
@McPizza0 McPizza0 deleted the feat/convoViewer branch February 2, 2024 17:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cstrnt cstrnt cstrnt approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@McPizza0 @cstrnt