Skip to content

Link to broadcast page when courseid == siteid on qm.php (#130) - #131

Open
djarran wants to merge 1 commit into
lsuonline:developfrom
catalyst:issue130-develop-upstream
Open

Link to broadcast page when courseid == siteid on qm.php (#130)#131
djarran wants to merge 1 commit into
lsuonline:developfrom
catalyst:issue130-develop-upstream

Conversation

@djarran

@djarran djarran commented Jul 7, 2026

Copy link
Copy Markdown

Description: Resolves #130. This PR aligns the block content and qm.php so that the "Compose message" links to broadcast.php when on the site course (ID = 1).

Testing steps:

  1. Install plugin

Test existing block functionality

  1. Visit course/view.php?id=1
  2. Add the Quickmail block
  3. Confirm that the Compose message button links to broadcast.php
  4. Create or visit a course
  5. Add the Quickmail block
  6. Confirm that the Compose message button links to compose.php

Test that qm.php now has same behaviour as block functionality regarding "Compose message" button

  1. Visit blocks/quickmail/qm.php?courseid=1
  2. Confirm that the Compose message button links to broadcast.php
  3. Visit blocks/quickmail/qm.php?courseid={courseid}
  4. Confirm that the Compose message button now links to compose.php

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.

1 participant