Skip to content
This repository was archived by the owner on Jul 17, 2026. It is now read-only.

Additional checks on permissions - #365

Merged
brianhogg merged 8 commits into
devfrom
fix/rest-api-checks
Jun 4, 2026
Merged

Additional checks on permissions#365
brianhogg merged 8 commits into
devfrom
fix/rest-api-checks

Conversation

@brianhogg

Copy link
Copy Markdown
Contributor

Description

Additional checks on a couple of the REST API endpoints.

How has this been tested?

Manually + PHPUnit tests

Checklist:

  • This PR requires and contains at least one changelog file.
  • My code has been tested.
  • My code passes all existing automated tests.
  • My code follows the LifterLMS Coding & Documentation Standards.

@brianhogg
brianhogg requested a review from ideadude as a code owner June 4, 2026 13:17
@brianhogg brianhogg moved this to Awaiting Review in Development Jun 4, 2026

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Scan Summary

Tool Critical High Medium Low Status
Dependency Scan (universal) 0 2 0 0
Secrets Audit 0 4 0 0
Shell Script Analysis 0 0 0 0

Recommendation

Please review the findings from Code scanning alerts before approving this pull request. You can also configure the build rules or add suppressions to customize this bot 👍

@brianhogg
brianhogg merged commit 2dfb111 into dev Jun 4, 2026
17 of 23 checks passed
@github-project-automation github-project-automation Bot moved this from Awaiting Review to Done in Development Jun 4, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant