https://twill.io logo
Join Discord
Powered by
# ❓questions
  • x
    public function messages() { $fields = [ 'color' => 'required|string', 'hide_menu' => 'boolean', 'hide_cta' => 'boolean', 'unlink_logo' => 'boolean', 'title' => 'required|string|max:200', ]; $messages = [ 'title.required' => "The title is required", 'color.required' => "Color is required" ]; return $this->messagesForTranslatedFields($messages, $fields); }
  • x
    dont work 😕
  • i
    that's not how it works
  • i
    the first parameter is messages for non translated rules, second is for translated rules, exactly like you did with the other helper above
  • x
    okay ... you mean the missing "required" tag at HTML form field in Twill generated form fields? I see. 😕
  • v
    Don't mind me, just try to do what ifox is saying
  • x
    okay... why the parameter is named $fields? you mean it so:
  • x
    public function messages() { $notTranslatedMessages = [ 'color.required' => "Color is required" ]; $translatedMessages = [ 'title.required' => "The title is required", ]; return $this->messagesForTranslatedFields($notTranslatedMessages, $translatedMessages); }
  • i
    yes, that should work. if not, there's definitely a bug
  • x
    unfortunately not
  • i
    can you check the response in your network tab/console?
  • x
    oh...there it is: {"message":"The given data was invalid.","errors":{"color":["Color is required"]}}
  • x
    hhmm...custom rules also dont work with the message at the field. in twill 2.1.1 it works, in 2.4 dont .. probably its a new bug with 2.4?
  • i
    definitely sound like a regression, would you mind reporting on GitHub issues so that we can keep track of it? thanks, and sorry for the trouble...
  • x
    sure, i do 🙂
  • j
    @User it's working for me with this code
  • j
    Copy code
    php
public function rulesForUpdate()
{
    return $this->rulesForTranslatedFields([
        'start_date' => 'required',
    ], [
        'url' => 'required',
    ]);
}
public function messages()
{
    return $this->messagesForTranslatedFields([
        'start_date.required' => 'custom date required message',
    ], [
        'url.required' => 'custom translated URL required message',
    ]);
}
  • x
    oh! with Twill 2.4?
  • j
    yes
  • x
    aahh .. I found my mistake. in our project we have in twill.php "bind_exception_handler = false" without the twill Handler. Thats it. Thanks to all of you!
  • b
    Having trouble with repeaters. I have a menu model and a menuItem model. The menu model hasMany menuItems and menuItems hasOne menu. There's already a menus table, a menu_items table, and a pivot table. However, when following the thread here: https://github.com/area17/twill/issues/131 I get some foreign key errors when trying to do pretty much anything
  • b
    Adding the override functions to my MenuRepository is what triggers the error
  • b
    Is that github thread still relevant? It says so in the docs
  • p
    Hey @User , I think the thread can still be helpful but there was a little update to the docs recently (not yet online). Have a look : https://github.com/area17/twill/blob/2.x/docs/.sections/form-fields.md#repeater
  • p
    The example doesn't use a pivot table, just a foreign key on the repeater item table.
  • b
    oh awesome! Let me take a look
  • b
    nice!
  • b
    works
  • b
    🙏 thanks!
  • b
    so in my case menuItems can be nested. Would I use some of the same code to get nested menuItems to display?
1...575859...484Latest