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: Implement notifications handling #25

Merged
merged 69 commits into from
Apr 7, 2024
Merged

feat: Implement notifications handling #25

merged 69 commits into from
Apr 7, 2024

Conversation

criss8X
Copy link
Collaborator

@criss8X criss8X commented Mar 27, 2024 •
edited
Loading

Aquí iré gestionando todo lo referente a tratado de notificaciones, obtención, guardado y eliminación de notificaciones.

Todavía seguimos gestionando más eventos, pero los ya gestionados pueden irse revisando.

criss8X added 29 commits March 20, 2024 13:48
@criss8X
feat: Implement ´DirectMessageModal´ component
068b8ba
@criss8X
feat: Abstract logic into ´useUserSearch´ and ´useInvitationLink´ hooks
807f575
@criss8X
feat: Implement ´DirectChatRecent´ component
936c15a
@criss8X
feat: Implement ´DirectMessageModal´ for SidebarActions
d2760c4
@criss8X
fix: Resolve comments of the PR
bd926fa
@criss8X
feat: Implement useDebounced hook
6531016
@criss8X
feat: More use useCallback for useSidebarActions
9f737fa
@criss8X
refactor: Convert SidebarActions as smart component
c693252
@criss8X
refactor: Improve Modal component for use createPortal
f38455e
@criss8X
Merge branch 'dev' into handle-sidebar-actions
21fd6e4
@criss8X
refactor: More use Modal component
299a79f
@criss8X
refactor: Improve modals handling for SidebarActions component
207ee5b
@criss8X
feat: Implement NotificationsModal and Notification components
1592310
@criss8X
fix: Fix visual problems for Modal
9b384bd
@criss8X
refactor: Resolve PR comments about tailwindcss class
83043cf
@criss8X
refactor: Remove redundant tailwindcss class
795372b
@criss8X
feat: Implement useNotification hook
74cc8c3
@criss8X
refactor: Resolve PR comments.
d4e1b66
@criss8X
refactor: Fix TailwindCss class problems
ad36140
@criss8X
feat: Implement notifications handling for useNotifications hook
058e07d
@criss8X
feat: Implement useSidebarModalActiveStore
53316ce
@criss8X
fix: Fix visual problems for components.
b6b5380
@criss8X
fix: Fix storybook stories problems
953f26a
@criss8X
feat: Upgrade Typography component
51752ff
@criss8X
feat: Upgrade UserProfile component
bf0da79
@criss8X
refactor: Refactor storybook stories
b429002
@criss8X
feat: Implement useChatInput for ChatContainer component
887aa81
@criss8X
fix: Fix visual problem for TextMessage
4d25b3e
@criss8X
feat: Implement useCachedNotifications hook
abd0dda
@criss8X criss8X requested a review from yurixander as a code owner March 27, 2024 01:19
@criss8X criss8X requested a review from yurixander April 2, 2024 19:02
yurixander
yurixander requested changes Apr 2, 2024
View reviewed changes
.github/workflows/deploy-pr-dev.yml Outdated Show resolved Hide resolved
src/components/SidebarActions/SidebarActions.tsx Outdated Show resolved Hide resolved
src/components/SidebarActions/useCachedNotifications.ts Outdated Show resolved Hide resolved
src/components/SidebarActions/useNotifications.ts Outdated Show resolved Hide resolved
src/components/SidebarActions/useNotifications.ts Outdated Show resolved Hide resolved
src/components/SidebarActions/useNotifications.ts Outdated Show resolved Hide resolved
src/components/SidebarActions/useNotifications.ts Outdated Show resolved Hide resolved
src/components/SidebarActions/useNotifications.ts Outdated Show resolved Hide resolved
src/components/SidebarActions/useNotifications.ts Outdated Show resolved Hide resolved
src/components/SidebarActions/useNotifications.ts Outdated Show resolved Hide resolved
@criss8X criss8X requested a review from yurixander April 3, 2024 15:37
yurixander
yurixander requested changes Apr 5, 2024
View reviewed changes
src/components/SidebarActions/useCachedNotifications.ts Outdated Show resolved Hide resolved
src/components/SidebarActions/useNotifications.ts Outdated Show resolved Hide resolved
src/hooks/matrix/useGlobalEventListeners.ts Outdated Show resolved Hide resolved
src/hooks/matrix/useGlobalEventListeners.ts Outdated Show resolved Hide resolved
src/utils/util.ts Outdated Show resolved Hide resolved
src/utils/util.ts Show resolved Hide resolved
@criss8X criss8X requested a review from yurixander April 5, 2024 18:44
yurixander
yurixander requested changes Apr 5, 2024
View reviewed changes
Copy link
Owner

@yurixander yurixander left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unas cositas de estilo de código y listo

src/components/SidebarActions/Notification.tsx Outdated Show resolved Hide resolved
src/components/SidebarActions/Notification.tsx Outdated Show resolved Hide resolved
src/components/SidebarActions/NotificationsModal.tsx Outdated Show resolved Hide resolved
src/components/SidebarActions/NotificationsModal.tsx Outdated Show resolved Hide resolved
src/hooks/matrix/useGlobalEventListeners.ts Outdated Show resolved Hide resolved
src/hooks/matrix/useGlobalEventListeners.ts Outdated Show resolved Hide resolved
src/hooks/matrix/useGlobalEventListeners.ts Outdated Show resolved Hide resolved
src/utils/util.ts Outdated Show resolved Hide resolved
@criss8X criss8X requested a review from yurixander April 6, 2024 04:34
yurixander
yurixander requested changes Apr 6, 2024
View reviewed changes
src/components/SidebarActions/NotificationsModal.tsx Outdated Show resolved Hide resolved
src/containers/ChatContainer/ChatContainer.tsx Outdated Show resolved Hide resolved
src/hooks/matrix/useActiveRoom.ts Show resolved Hide resolved
src/hooks/matrix/useGlobalEventListeners.ts Outdated Show resolved Hide resolved
src/utils/notifications.ts Outdated Show resolved Hide resolved
src/utils/util.ts Outdated Show resolved Hide resolved
@criss8X criss8X requested a review from yurixander April 6, 2024 14:33
@criss8X criss8X merged commit 9344de6 into dev Apr 7, 2024
5 checks passed
@criss8X criss8X deleted the cristian/handle-notifications branch April 21, 2024 15:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lazaroysr96 lazaroysr96 lazaroysr96 approved these changes

@yurixander yurixander yurixander approved these changes

Assignees

@criss8X criss8X

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@criss8X @yurixander @lazaroysr96