Helper for CheckboxSelectMultiple widget which allows user to select/… #879
Conversation
…unselect all available options.
| # tty: true | ||
| # Following allows to execute `docker attach feder_web_1` | ||
| # necessary when using ipdb in development: | ||
| stdin_open: true |
There was a problem hiding this comment.
I suggest comment out that back. It's out of scope of that PR.
There was a problem hiding this comment.
I know, it's out of scope but it's just about to be merged into master. I think it's useful to save some time. To be honest, this super correctness makes me work much slower and I don't like the idea of making separate branch just for things so small. I mean, there is nothing unclear in this case, which would lead to any problems in my opinion.
There was a problem hiding this comment.
Ja skomentowałem, bo odniosłem wrażenie, że zacommitowałeś odkomentowane przypadkiem. Jak wprowadzałeś zacommitowane to się nie odezwałem.
There was a problem hiding this comment.
Rozumiem, w tym przypadku akurat celowo odkomentowałem, bo pomyślałem, że tak będzie wygodniej dla wszystkich deweloperów. Raczej niczemu to nie szkodzi, bo Dockera mamy tylko dla developmentu, a ułatwia debug. Chyba, że o czymś nie wiem?
There was a problem hiding this comment.
Jak ułatwia i testy przechodzą to niech będzie.
| @@ -556,8 +556,7 @@ | |||
| "colorette": { | |||
| "version": "1.2.1", | |||
| "resolved": "https://registry.npmjs.org/colorette/-/colorette-1.2.1.tgz", | |||
| "integrity": "sha512-puCDz0CzydiSYOrnXpz/PKd69zRrribezjtE9yd4zvytoRc8+RY/KJPvtPFKZS3E3wP6neGyMe0vOTlHO5L3Pw==", | |||
| "dev": true | |||
| "integrity": "sha512-puCDz0CzydiSYOrnXpz/PKd69zRrribezjtE9yd4zvytoRc8+RY/KJPvtPFKZS3E3wP6neGyMe0vOTlHO5L3Pw==" | |||
There was a problem hiding this comment.
Faktycznie, musiałem docommitować.
…unselect all available options.