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

Development/notification #2110

Open
wants to merge 24 commits into
base: master
Choose a base branch
from
Open

Development/notification #2110

wants to merge 24 commits into from
+1,135 −182

Conversation

yacineKahlerras
Copy link
Contributor

  • notification popover
  • notification page

Enjoy2Live and others added 24 commits July 19, 2024 17:41
@Enjoy2Live
start working on notification popover
90e16d6
@Enjoy2Live
Add notifications route to the Nextjs app
d17f8f1
@yacineKahlerras
init
0b2ae0e
@yacineKahlerras
remobed props
3b456bd
@yacineKahlerras
getting the profile img back to the user's
60c2988
@Enjoy2Live
remove global styles from website except learn and create pages
b780d7d
@yacineKahlerras
added notification item in a li
f49a272
@yacineKahlerras
init
9b21e8a
@yacineKahlerras
remobed props
beb7be0
@yacineKahlerras
getting the profile img back to the user's
9c7f34d
@yacineKahlerras
added notification item in a li
4bce4c1
@yacineKahlerras
Merge branch 'yacine/notification-popover-#1667' of github.com:dev-la…
da3032f 
…unchers/dev-launchers-platform into yacine/notification-popover-#1667
@yacineKahlerras
fixed some issues with the pr
146c2c0
@yacineKahlerras
added transtack query
a4c7cad
@Enjoy2Live
install react query
fb2221f
@yacineKahlerras
added function to agent and fetching data with transtack query
b2c7aca
@yacineKahlerras
added some custom stuff for mobile in the notification popover
f7d739f
@yacineKahlerras
added scroll behaviour and implemented some mock data
807c20c
@yacineKahlerras
calculating timestap for notifications
38cd68f
@yacineKahlerras
bug fix and mobile design
411959c
@yacineKahlerras
removing unnecessary console log
877d189
@yacineKahlerras
Merge pull request #1890 from dev-launchers/yacine/notification-popov…
6f3fb10 
…er-#1667

added the notification popover
@yacineKahlerras
updated notification requirements
7d873d8
@yacineKahlerras
conent visible now
79f4a2f
@changeset-bot changeset-bot
Copy link

changeset-bot bot commented Nov 2, 2024

⚠️ No Changeset found

Latest commit: 79f4a2f

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

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

@Enjoy2Live Enjoy2Live Awaiting requested review from Enjoy2Live Enjoy2Live is a code owner

@pyxld-kris pyxld-kris Awaiting requested review from pyxld-kris pyxld-kris is a code owner

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.
2 participants
@yacineKahlerras @Enjoy2Live