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

Fe only #3

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

Fe only #3

wants to merge 54 commits into from

Conversation

ihabzee
Copy link

@ihabzee ihabzee commented Apr 30, 2024

No description provided.

Jordan-M-Schultz and others added 30 commits January 10, 2024 20:02
@Jordan-M-Schultz
Refactored refresh from server, fixed bug stacking interval callbacks
c78c6cd
@Jordan-M-Schultz
Added predis package, created test page
0972870
@ihabzee
add notification cache factory.
1f7a30c
@ihabzee
add notification cache factory.
aefd53d
@ihabzee
change logic
ff80442
@Jordan-M-Schultz
Added predis to CacheInterface, test page
8398f48
@ihabzee
add generate key method.
8245700
@ihabzee
Merge branch 'redis_config' of github.com:susom/redcap_notifications …
29fdf7c 
…into redis_config
@Jordan-M-Schultz
Cleaned up factory, removed unnecessary variables
7b8fbb8
@Jordan-M-Schultz
Added setKeys and getKeys functions
cf68743
@Jordan-M-Schultz
Added setKeys and getKeys functions
44ab197
@Jordan-M-Schultz
Added delete and list keys
d5dc488
@Jordan-M-Schultz
Added function calls for key operations
5abda92
@Jordan-M-Schultz
Merge branch 'redis_config' of https://github.com/susom/redcap_notifi…
3a22800 
…cations into redis_config
@ihabzee
add few methods for cache keys.
b1bca38
@Jordan-M-Schultz
Removed extraneous pages from config
6327227
@ihabzee
add new method to pull notifications.
bf65e6b
@ihabzee
commit config.json
ce4ddec
@ihabzee
implement getNotifications method.
3c84e59
@ihabzee
fix datatype error.
e4f7137
@ihabzee
add notification dismissal
1fb17dc
@irvins
merge from redis_config
a3a1a3f
@Jordan-M-Schultz
merging ui and api changes
de32c98
@irvins
removing uneccesary files
c1dc013
@irvins
woops forgot to remove the include line
8d700bf
@ihabzee
remove cache from config.json
ccb6cf9
@ihabzee
clean config.json
2e39461
@Jordan-M-Schultz
Retooling javascript to work with new backend
507041a
@Jordan-M-Schultz
Retooling javascript to work with new backend x2
d440e99
@irvins
adjusting the dimsiss_notif and cleaning up some client side functio…
774ed95 
…ns no longer needed... may need to do some adjustements to the function resolveDismissed() depending on what is returned
ihabzee and others added 24 commits February 13, 2024 14:59
@ihabzee
fix minor bug in dismiss endpoint.
6d98540
@irvins
removing unecceary dismiss checks and carrying over snooze_expire
e57ee96
@irvins
rmeoiveng a debug line and using the callAjax2 method
a7fc427
@irvins
added a default value for snooze_expire
07d6b00
@ihabzee
add mysql lock to prevent duplicate user dismiss key.
24f354d 
fix minor bug.
@irvins
dismiss all ajax was firing too quick , so added small delay...now it…
5bd44ff 
… works as expected
@irvins
t Merge branch 'fe_only' of github.com:susom/redcap_notifications int…
d49485d 
…o fe_only
@irvins
hide dismiss_all button if no notifs are dismisssable after an actio…
6c39824 
…n, hide notif UI for banner and modal if no notifs are left after action
@irvins
adding css so that the modal will grow only up to a fixed height, an…
2115dec 
…d then introduce an inline vertical scroll after that
@irvins
fixing CSS for home / survey page display , and adding height limit …
ac8b896 
…for banners
@ihabzee
fix html component.
a62e4bb
@ihabzee
Merge branch 'fe_only' of github.com:susom/redcap_notifications into …
f0eefd0 
…fe_only
@irvins
putting all of them into the same container so the maximum expansion…
7c970f0 
… will be just one, or do we need them to be grouped by type?
@irvins
Merge branch 'fe_only' of github.com:susom/redcap_notifications into …
7c1f461 
…fe_only
@ihabzee
when notification is loaded log that it has been viewed.
98e665c
@irvins
updating the skull and crossboanes default danger icon
ab4657e
@ihabzee
Merge branch 'fe_only' of github.com:susom/redcap_notifications into …
b0b1d46 
…fe_only
@ihabzee
change get project id to return array of pids instead of one.
baca386
@ihabzee
add unit tests
b8e3028
@ihabzee
do not create notification viewed log if no notification exists.
6d22e8a
@ihabzee
fix minor php errors.
71022b5
@ihabzee
fix for notifications project status.
35960ae
@ihabzee
new update project status EM to automatically update project status i…
ddf14ef 
…f days elapsed.
@ihabzee
update log message.
69ab909
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ihabzee @Jordan-M-Schultz @irvins