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 multi-select and keyed options support #12

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

frumbert
Copy link

I need the drop-down field type to be able to store items in two extra configurations :

  • select-multiple, where I can choose more than one option
  • different key stored in db than is displayed (so that the moodle form renders as <option value="my_value">The Text</option>)

This pull-request adds both these to the menu field type and adds a couple of entries to the lang file to explain the format (might have botched the lang file and version though)

Added new event type for emitting details of instance metadata apon save
@frumbert
Copy link
Author

frumbert commented Aug 2, 2018

Added event trigger handler, fixed incorrectly saved data value for menu items in previous commit

@mchurchward
Copy link
Contributor

Hi @frumbert. Can you pull the latest changes into your local repo and regenerate this pull request? If there is no database changes, don't bother with the version file.

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.

2 participants