Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Improve help text for integration of icons (manual trigger) #193

Merged
merged 2 commits into from
Jan 16, 2024
Merged

Improve help text for integration of icons (manual trigger) #193

merged 2 commits into from
Jan 16, 2024

Conversation

KerstinSc
Copy link
Contributor

Hi!

This should solve #176 . The help text was a little bit misleading because it produced an error (icon not showing up). I'm not sure if moodle had recently changed something regarding the integration of the icons.

Kind regards!

@NinaHerrmann
Copy link
Contributor

Works for 4.1 - not sure if there was a change but in case there was one it is in an outdated version.
Thanks for the contribution!

@NinaHerrmann NinaHerrmann merged commit 38d6dc7 into learnweb:master Jan 16, 2024
@KerstinSc KerstinSc deleted the improve-help-text branch January 16, 2024 14:20
NinaHerrmann added a commit that referenced this pull request Jun 19, 2024
* Removed ,

* CI: Update for Moodle 4.2

* Workflow backup: Do not save attributes when they are null

* Bump version

* MBS-7377: Fix workflowoverview for big instances

* Fix wrong calculation of courses counts in template context

* fix upload workflow (#185)

* fix upload workflow

* inserting feedback

* finally

* Add check if process exists. (#191)

This prevents courses to be processed which had been rescued by the teacher.

* Update README.md

Make Subplugin List (and link to wiki) more apparent

* Improve help text for integration of icons (manual trigger) (#193)

* Improved help text for the integration of icons

* Improved help text for the integration of icons

* Update for Moodle 4.3 (#190)

* CI: Update for Moodle 4.3

* Prevent dynamic property access on classes for PHP 8.2

* phpcbf for styles etc.

---------

Co-authored-by: Justus Dieckmann <justusdieckmann@wwu.de>
Co-authored-by: NinaHerrmann <nina.herrmann@uni-muenster.de>

* Hide menu when there is no manual workflow (#195)

Co-authored-by: Justus Dieckmann <justusdieckmann@wwu.de>

* Fix step and trigger reordering: only change the relevant workflow (#200)

* Reorder the remaining active workflows when disabling a workflow (#201)

This fixes #128.

* Show menu entry when a manual or automatic workflow is active (#199)

* Show menu point when a manual or automatic workflow is active

* Rename reset cache function

* Show menu entry if there are workflows other than the default two

---------

Co-authored-by: Justus Dieckmann <justusdieckmann@wwu.de>

* Check backup consistency before importing workflow (#196)

* Check backup consistency before importing workflow

* Fix PHPDocs

* Use existing category in backup_and_restore test

* lifecycletrigger_categories: A bit of general refactoring

* lifecycletrigger_categories: Don't fail if category doesn't exist

* Add possibility to force import of errorneous backup

* Use $force = false as default in restore_workflow:execute

* Display only unique errors when Workflows causes multiple errors

---------

Co-authored-by: Justus Dieckmann <justusdieckmann@wwu.de>
Co-authored-by: NinaHerrmann <nina.herrmann@uni-muenster.de>

* CI: Update for Moodle 4.4

* Codestyle PHPCBF

* Sort strings in lang files alphabetically

* Fix sortindex gaps in upgrade step

* Bump version

* The columns in active_manual_workflows_table shouldn't be sortable

* Shell escape the release notes in the release workflow (#207)

Co-authored-by: Justus Dieckmann <justusdieckmann@wwu.de>

* Fix mustache

* added mtrace statement for logging failed mails (#214)

* Shell escape release workflow for realsies this time (#208)

---------

Co-authored-by: Justus Dieckmann <justusdieckmann@wwu.de>
Co-authored-by: Philipp Memmel <philipp.memmel@gmail.com>
Co-authored-by: Justus Dieckmann <45795270+justusdieckmann@users.noreply.github.com>
Co-authored-by: Melanie Treitinger <66598405+melanietreitinger@users.noreply.github.com>
Co-authored-by: KerstinSc <kerstin.schmidt@th-nuernberg.de>
Co-authored-by: Kata <35190838+olivabigyo@users.noreply.github.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants