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.

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

Add 'urlonly' Target Option for LTI Shortcode #8

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

abdolnabi
Copy link

@abdolnabi abdolnabi commented Aug 15, 2024

Fixes #7
This pull request introduces a new 'urlonly' target option for LTI shortcodes within the wordpress-lti-platform. This option allows users to generate only the URL without embedding or linking directly, facilitating easier integration with custom UI components.

Changes:

  1. class-lti-platform-tool.php: Extended the shortcode handler to include 'urlonly' as a target option. This modification enables the shortcode to return just the URL when 'urlonly' is specified.
  2. lti-platform-admin-edit.php: Updated the LTI Platform Settings menu in the admin panel to include 'urlonly' in the dropdown for presentation targets. This update allows administrators to easily select this new option from the platform settings.
  3. class-lti-platform-admin.php: Updated the LTI Platform Settings menu in the admin panel to include 'urlonly' in the dropdown for presentation targets. This update allows administrators to easily select this new option from the platform settings.
  4. public/class-lti-platform-public.php: Updated the LTI Platform Settings menu in the admin panel to include 'urlonly' in the dropdown for presentation targets. This update allows administrators to easily select this new option from the platform settings.

Motivation:
The addition of the 'urlonly' option enhances flexibility for developers and admins, allowing more streamlined integration of LTI tools within custom solutions not requiring embedded or directly linked content.

Testing:

  • Manual tests were conducted to ensure that selecting 'urlonly' correctly outputs only the URL.
  • Verified that existing functionality for other target options remains unaffected.

Please review the changes and merge them if they meet the project standards.

- Updated class-lti-platform-tool.php to include 'urlonly' in shortcode processing.
- Modified lti-platform-admin-edit.php to allow 'urlonly' selection in the LTI Platform Settings UI.
- Updated class-lti-platform-tool.php to include 'urlonly' in shortcode processing.
- Modified lti-platform-admin-edit.php to allow 'urlonly' selection in the LTI Platform Settings UI.
@abdolnabi abdolnabi marked this pull request as draft August 15, 2024 08:05
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.

Add URL Only Output Option to Shortcode in WordPress LTI Platform
1 participant