Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • Jan 28 2019 22:15

    bobvandevijver on master

    Fixed documentation indentation… (compare)

  • Jan 28 2019 22:13

    bobvandevijver on master

    Fixed documentation indentation… (compare)

  • Dec 18 2018 14:45
  • Aug 29 2018 12:36
  • Aug 28 2018 06:49
  • Aug 01 2018 18:20

    tobias-93 on v2.5.1

    (compare)

  • Aug 01 2018 18:18

    tobias-93 on master

    Add choice_translation_domain t… (compare)

  • Jul 17 2018 05:55
  • Jun 26 2018 12:41
    snuffer82 commented #322
  • Jun 26 2018 08:38
    bobvandevijver closed #322
  • Jun 26 2018 08:38
    bobvandevijver commented #322
  • Jun 12 2018 09:50
    snuffer82 edited #322
  • Jun 12 2018 09:40
    snuffer82 opened #322
  • May 31 2018 08:38
    bobvandevijver closed #321
  • May 31 2018 08:38
    bobvandevijver commented #321
  • May 30 2018 18:45
    bobvandevijver labeled #321
  • May 30 2018 18:45
    bobvandevijver opened #321
  • May 28 2018 15:52

    bobvandevijver on v2.5.1

    (compare)

  • May 28 2018 15:43

    bobvandevijver on v2.5.1

    (compare)

  • May 28 2018 15:36

    bobvandevijver on 2.5.0

    (compare)

Bob van de Vijver
@bobvandevijver
Good call, but there is probably also a way around that :)
Tobias Feijten
@tobias-93
@sescandell any thoughts on #226 ?
Stéphane
@sescandell
Well, if return method_exists(AbstractType::class, 'getBlockPrefix') ? FormType::class : 'form'; in the getExtendedType() is the only thing that needs to be added, I would say go for it
actually, it's a little bit more than that: form aliases, using form type, ...

with "a little" effort

This is not anymore valid :)... it might be annoying

Stéphane
@sescandell
I know I introduced many BCBreak fo 2.8 and 3.0 compatibility. But the thing is: all these bundles are not (yet) released: I think we could so make these BC Break. I'm open to any discussion, let me know if you think all of this IS annoying
My objectives are to validate 2.8 & 3.0 compatibilty
then create released tags
(and bugfix / complete the documentation then)
Bob van de Vijver
@bobvandevijver
I'm okay with that
If anyone does not use 2.8 or higher yet, than they should fix that or use an older tag
Stéphane
@sescandell
:+1:
ioleo
@ioleo
Check out my new project https://github.com/loostro/cryptomute :) Travis tests are still failing, but only becouse I don't know how to install php_gmp on 5.4 and 5.5 (on travis).
@sescandell @bobvandevijver maybe you know how to install php_gmp on php 5.4 and 5.5?
Bob van de Vijver
@bobvandevijver
apt-get install php5-gmp?
Not sure how that works on Travis though
Bob van de Vijver
@bobvandevijver
Oh, it is an old ubuntu and with the apt-get you do install it, but for the wrong version...
ioleo
@ioleo
ah.. i thought so
there is some kind of php version manager there
You'll need to use PECL
ioleo
@ioleo
i tried, but GMP is not in PECL repositories :(
Karel Sommer
@calvera
hello! my question is simple, can I have two excel reports for one list with different columns? Is there a simple way?
Bob van de Vijver
@bobvandevijver
Well, there is no simple way
You can off course overwrite the methods based on the request, but it is not configurable through the generator file
Karel Sommer
@calvera
yes, that is the way, I could use generator to prepare base ExcelController and save it, then change yaml file and generate second one... thanks
Serg N. Kalachev
@ksn135

@calvera @bobvandevijver Hi there ! I have a similar case. May be we should extend admin generator for such cases?
Current implementation (left untouched):

builders:
    excel:
        params: 
            credentials: 'hasRole("ROLE_SUPER_ADMIN")'        
            display:
                - id
                - guid
                - dirty
                - textState
                - kind
                - fullName

New featured "multi-report mode":

builders:
    excel:
        params: 
            credentials: 'hasRole("ROLE_SUPER_ADMIN")'        
            display:
               "Full report":
                  - id
                  - guid
                  - dirty
                  - textState
                  - kind
                  - fullName
               "Short report":
                  - id
                  - fullName
                  - textState
               "Special report":
                  - id
                  - guid
                  - dirty

What do you think about it ?

I probably could make a PR for this
Bob van de Vijver
@bobvandevijver
If we want to support it, we should also think about different credentials
So that powerusers can get a full report, and other users a special report
Or something like that
Serg N. Kalachev
@ksn135
Yep! of course!
Many things to discuss
May be we should draw a dropdown menu with all available reports for current user
Add don't forget different fa- icons
Bob van de Vijver
@bobvandevijver
And a single button if there is only one option
Serg N. Kalachev
@ksn135
you read my mind )
Bob van de Vijver
@bobvandevijver
Haha :)
And here I was, wondering how many people would even use the Excel export :smile:
Serg N. Kalachev
@ksn135
at least five! that's enough for me )
Bob van de Vijver
@bobvandevijver
Do we also want different label for the fields in the different exports?
I do not have time to create anything like this in the near future, so if you think that you can create the functionality, you're welcome @ksn135 :)
Maybe first open an issue, so that we can discuss
Serg N. Kalachev
@ksn135
Done. #262
Bob van de Vijver
@bobvandevijver
Thanks
Karel Sommer
@calvera
+1 to this!
Bob van de Vijver
@bobvandevijver
@sescandell I'm having a problem with the admingenerator relying on the iCheck dependency, which I do not use. It's about two rules in our main.js file: main.js#L79 & main.js#L109.
On line 79, we can simply change the on('ifChecked ifUnchecked' to on( 'change', but I'm not sure if the .iCheck('update') is needed
Guess I can also make an check for it
Bob van de Vijver
@bobvandevijver
Hmm, not even sure why we have the dependency on iCheck, it's the only place it's used..