https://twill.io logo
Join Discord
Powered by
# ❓questions
  • t

    tfilos

    09/20/2022, 7:28 PM
    Output
    config('twill')
    in tinker and I get
    Copy code
    "media_library" => [
           "disk" => "twill_media_library",
           "endpoint_type" => "local",
           "cascade_delete" => false,
           "local_path" => "uploads",
           "image_service" => "A17\Twill\Services\MediaLibrary\Glide",
  • t

    tfilos

    09/20/2022, 7:28 PM
    So there are no surprises there.
  • t

    tfilos

    09/20/2022, 7:28 PM
    Copy code
    "file_library" => [
           "disk" => "twill_file_library",
           "endpoint_type" => "local",
           "cascade_delete" => false,
           "local_path" => "uploads",
           "file_service" => "A17\Twill\Services\FileLibrary\Disk",
           "acl" => "public-read",
           "filesize_limit" => 50,
           "allowed_extensions" => [],
           "prefix_uuid_with_local_path" => false,
         ],
  • t

    tfilos

    09/20/2022, 7:28 PM
    All looking correct
  • t

    tfilos

    09/20/2022, 7:36 PM
    Since that stuff all looks correct, I'm thinking this might be a coincidence and not happening from the Twill end of things
  • t

    tfilos

    09/20/2022, 7:40 PM
    Will keep digging. Thanks!
  • i

    ifox

    09/20/2022, 7:46 PM
    interesting, definitely let us know what you find but yeah right now I can't think of anything else in Twill that would trigger this.
  • c

    chin

    09/20/2022, 9:58 PM
    Is it possible to check for the current authenticated admin user from a basic route like web.php?
    Copy code
    auth('twill_users')->user()
  • i

    ifox

    09/20/2022, 10:05 PM
    Auth::guard('twill_users')->check()
  • i

    ifox

    09/20/2022, 10:05 PM
    ha you got it
  • c

    chin

    09/20/2022, 10:05 PM
    haha yeah found it here! https://spectrum.chat/twill/help/check-if-twill-user-is-logged-in-or-not~da6ee207-c3ef-4266-927c-4ea3fa686388 thanks for the quick reply 🙂
  • b

    BA7YA

    09/21/2022, 7:33 AM
    Hi guys. On which step twill is calling this method? https://github.com/area17/twill/blob/c1a0f4f8e259a07fad5c8bec101c6c4f97599fd2/src/Repositories/Behaviors/HandleRevisions.php#L42 Can you show me the place
  • i

    ifox

    09/21/2022, 7:39 AM
    https://github.com/area17/twill/blob/c1a0f4f8e259a07fad5c8bec101c6c4f97599fd2/src/Repositories/ModuleRepository.php#L287 which calls https://github.com/area17/twill/blob/c1a0f4f8e259a07fad5c8bec101c6c4f97599fd2/src/Repositories/ModuleRepository.php#L605
  • b

    BA7YA

    09/21/2022, 7:47 AM
    About a problem with not working revisions I found strange behaviour. I put
    dd($this->traitsMethods(__FUNCTION__))
    in this method https://github.com/area17/twill/blob/c1a0f4f8e259a07fad5c8bec101c6c4f97599fd2/src/Repositories/ModuleRepository.php#L605. 1st screenshot when Im creating the item in module where revisions are working. when Im creating the item in module where revisions are **NOT **working, twill even not calling that beforeSave method.
  • b

    BA7YA

    09/21/2022, 7:53 AM
    Yeah. It's just empty $this->traitsMethods
  • i

    ifox

    09/21/2022, 7:58 AM
    I mean if this is empty it means the class doesn't have traits
  • i

    ifox

    09/21/2022, 8:00 AM
    I know you said you had it but I can't see why php wouldn't see them
  • m

    mateoD

    09/21/2022, 10:28 AM
    Hello everyone , after I updated composer to get the version of twill from 2.8 to 2.9, it gives me this error
    Copy code
    Class people specified in twill.dashboard configuration does not exists.
  • i

    ifox

    09/21/2022, 10:32 AM
    hi @mateoD sorry about that. It seems like a PR we released in 2.9 is causing this for a few users
  • i

    ifox

    09/21/2022, 10:33 AM
    https://github.com/area17/twill/issues/1822
  • i

    ifox

    09/21/2022, 10:33 AM
    what is your morphmaps looking like?
  • i

    ifox

    09/21/2022, 10:36 AM
    @Harings Rob this commit specifically https://github.com/area17/twill/commit/5589c9731a5ff6826f3ad6f65ced962208e89b13
  • i

    ifox

    09/21/2022, 10:37 AM
    Seems like this is a regression as it would require to use the model FQCN instead of it's morphmaps shorthand
  • i

    ifox

    09/21/2022, 10:38 AM
    we'll address this and release 2.9.1 @mateoD
  • m

    mateoD

    09/21/2022, 10:41 AM
    thanks , but for now should go back to the previous version ?
  • i

    ifox

    09/21/2022, 10:42 AM
    we should be able to release this quickly, so you can either do that or remove modules from your dashboard configuration in the meantime
  • m

    mateoD

    09/21/2022, 10:47 AM
    ok thanks , i removed modules works for now. I believe that this method is not a problem for other functionalities or may cause any other problem ?
  • h

    Harings Rob

    09/21/2022, 10:50 AM
    I guess we just remove that exception?
  • h

    Harings Rob

    09/21/2022, 10:50 AM
    Or leave it but include the morphmap for checking. I did this elsewhere
  • h

    Harings Rob

    09/21/2022, 10:50 AM
    $moduleClassToCheck = Relation::getMorphedModel($moduleClass) ?? $moduleClass;
    I will make a hotfix now
1...401402403...484Latest