Skip to content

Conversation

@individual-it
Copy link
Collaborator

without this change only admins are allowed to use the endpoint, this does not make sense and lead to #256 (comment)

@kiranparajuli589
Copy link
Contributor

looks good. when einstein (non admin user) marks as read the notification from NC dashboard, the notification gets marked successfully in NC and synchronized in the OP. 👍

@kiranparajuli589 kiranparajuli589 changed the title allow now admin user to mark notifications as read allow non admin user to mark notifications as read Nov 2, 2022
Signed-off-by: Artur Neumann <artur@jankaritech.com>
@kiranparajuli589 kiranparajuli589 force-pushed the allowNonAdminToMarkNotificationAsRead branch from 0013caa to c3a016d Compare November 3, 2022 05:56
@kiranparajuli589
Copy link
Contributor

CI should be green now ✔️

@github-actions
Copy link

github-actions bot commented Nov 3, 2022

JS Code Coverage

Coverage after merging allowNonAdminToMarkNotificationAsRead into master will be
93.81%
Coverage Report
FileStmtsBranchesFuncsLinesUncovered Lines
src
   adminSettings.js0%0%0%0%1, 1, 10–19, 2, 20–25, 3–9
   bootstrap.js0%0%0%0%1, 1–7
   dashboard.js0%0%0%0%1, 1, 10–19, 2, 20–27, 3–9
   fileActions.js0%0%0%0%1, 1, 10–17, 2–9
   personalSettings.js0%0%0%0%1, 1, 10–19, 2, 20–25, 3–9
   projectTab.js0%0%0%0%1, 1, 10–19, 2, 20–29, 3, 30–39, 4, 40–49, 5, 50–59, 6, 60–66, 7–9
   utils.js57.45%33.33%50%59.52%10–14, 17–26, 6–9
src/components
   AdminSettings.vue99.08%95.83%77.78%99.86%1, 1, 1
   OAuthConnectButton.vue99.08%80%100%100%1
   PersonalSettings.vue97.81%91.67%71.43%99.39%1, 1
src/components/admin
   FieldValue.vue98.91%90%100%100%1
   FormHeading.vue98.98%75%100%100%1
   TextInput.vue99.27%94.12%87.50%100%1
src/components/icons
   ClippyIcon.vue93.18%50%50%97.50%1, 1
src/components/settings
   CheckBox.vue92.45%80%66.67%97.62%1, 1
   SettingsTitle.vue95.56%50%100%97.62%1, 1
src/components/tab
   EmptyContent.vue99.34%90.91%100%100%1
   SearchInput.vue99.59%83.33%100%100%1
   WorkPackage.vue99.01%33.33%100%99.66%1, 1, 1
src/utils
   workpackageHelper.js97.46%96%100%97.75%17–19
src/views
   Dashboard.vue98.20%50%50%99.63%1, 1, 1
   ProjectsTab.vue99.74%92.31%100%100%23

@github-actions
Copy link

github-actions bot commented Nov 3, 2022

PHP Code Coverage

Coverage after merging allowNonAdminToMarkNotificationAsRead into master will be
64.65%
Coverage Report
FileStmtsBranchesFuncsLinesUncovered Lines
lib/AppInfo
   Application.php20.69%100%25%20%106, 110, 61–62, 65, 72, 74–76, 81–84, 86, 90–95
lib/BackgroundJob
   CheckNotifications.php0%100%0%0%48, 50, 52–53, 61–62
lib/Controller
   ConfigController.php79.58%100%66.67%80.45%112–113, 115, 117–119, 121, 127–128, 130, 320–323, 325–326, 329, 337–341, 363–365, 94
   DirectDownloadController.php0%100%0%0%36–38, 53–55, 57, 60–61
   FilesController.php73.68%100%100%71.91%168, 221–222, 267, 273–277, 281–282, 285, 287, 298–300, 303–304, 306–307, 311–314, 317
   OpenProjectAPIController.php84.92%100%78.57%85.41%136, 171, 188–189, 193, 197, 199–204, 206, 215–216, 219, 221, 223–226, 228–229, 234, 253, 278, 95
lib/Dashboard
   OpenProjectWidget.php0%100%0%0%101, 108, 115–116, 118, 120–121, 123–124, 126, 129–130, 132–133, 135, 69–73, 80, 87, 94
lib/Exception
   OpenprojectErrorException.php100%100%100%100%
   OpenprojectResponseException.php100%100%100%100%
lib/Listener
   LoadSidebarScript.php0%100%0%0%100–101, 103–104, 65–70, 72–73, 75–76, 78–79, 85–86, 88, 90, 92–94, 96, 98
lib/Notification
   Notifier.php0%100%0%0%100–104, 109–110, 112, 114, 47–50, 60, 69, 80, 82, 85, 87–92, 95, 97
lib/Search
   OpenProjectSearchProvider.php0%100%0%0%102, 109–110, 113–116, 118–119, 123–125, 127–128, 130–131, 133–135, 138–139, 141–142, 146–147, 149–151, 157–158, 160, 169, 177–183, 192–197, 206, 71–75, 82, 89, 97, 99
   OpenProjectSearchResultEntry.php100%100%100%100%
lib/Service
   DirectDownloadService.php88%100%100%86.96%65–66, 68
   OauthService.php0%100%0%0%101–105, 115–118, 42–44, 53–59, 61, 70–73, 84–91
   OpenProjectAPIService.php88.49%100%92.31%88.13%179–183, 212–216, 373–374, 376, 390–391, 400, 404, 425, 513–514, 521, 524–527, 529, 535, 539–541, 713, 784, 815
lib/Settings
   Admin.php0%100%0%0%32–34, 41–43, 46–50, 53, 58–60, 63, 65–66, 68, 72, 76
   AdminSection.php0%100%0%0%19–20, 29, 39, 48, 55
   Personal.php88.89%100%50%93.75%101, 105
   PersonalSection.php0%100%0%0%19–20, 29, 39, 48, 55

@kiranparajuli589 kiranparajuli589 merged commit a1937b3 into master Nov 3, 2022
@delete-merged-branch delete-merged-branch bot deleted the allowNonAdminToMarkNotificationAsRead branch November 3, 2022 06:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants