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

Fix HandleFieldsGroups trait #452

Merged
merged 1 commit into from
Nov 14, 2019
Merged

Conversation

m4n1ok
Copy link
Member

@m4n1ok m4n1ok commented Nov 14, 2019

With #410, there are some regressions into Twill repositories which extend ModuleRepository.
If $fieldsGroups isn't define some repositories will break.
I added a default value for $fieldsGroups into ModuleRepository.

@m4n1ok m4n1ok requested review from ifox and yanhao-li November 14, 2019 13:02
@ifox ifox merged commit 657e83c into area17:1.2 Nov 14, 2019
@m4n1ok m4n1ok deleted the fix/HandleFieldsGroups branch November 14, 2019 14:39
m4n1ok added a commit to m4n1ok/twill that referenced this pull request Nov 14, 2019
* 1.2:
  Add style for subscript into wysiwyg (area17#450)
  Add a default value for `$fieldsGroups` to prevent error from repositories which extend ModuleRepository. (area17#452)
  fixed the http & https issue from APP URL (area17#431)
  Add a feature to group fields in a JSON column (area17#410)
  Replace bcrypt() with Hash::make in CreateSuperAdmin command (area17#445)
  Fixed the issue that 'delete' => true not working in indexOptions, refs area17#289 (area17#446)
  Fix « Handle Repeater » feature compatibility with Laravel 5.6 (area17#449)
  Revert "Add quotes when populating store from form field (area17#416)"
  Fix media and file libraries upload endpoint logic
  Properly support fileupload for Azure (area17#443)
  Invert sorting order in ModuleRepository.
  Support azure storage (area17#424)
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