Pierre Allix comments

Results 65 comments of


                                            Pierre Allix

showing standard features for logged-out users is hacky

Can we remove backend tag on this?

incomplete detection of references on deepPost (batch API)

@joka but for the short term, is checking for targetsheet okay? It seems to me that it is created for all sheet having a 'reftype', thus for all references?

Badge assignments options request returns options if there are none

> To check if the user can create a badge assignment and assign a badge we can add the list of assignable badges. Isn't it already the case?

Euth posting empty empty image

What happens if you post without the "picture" key? @joka is `null` allowed as a value for references?

Pool search at depth one but SearchQuery defaults to depth zero

This caused the #1140 bug when refactoring a migration script and omitting the depth.

Allow inheritance at the permission level of each action for workflows using 'default'

It does not work because Deny for Everyone takes precedence in the ACL of the resource at depth n over Allow for Participant in the ACL of resource at depth...

Allow inheritance at the permission level of each action for workflows using 'default'

I think it makes the logic a bit more complicate also, inclusive for the user, when one operator could have two different meaning. That said we do not have a...

Ensure complex protobuf files can be parsed

@bitwalker could you have a look and tell me what you think? the changes are small but important for my team

Ensure complex protobuf files can be parsed

@bitwalker Paul, any chance to get that into master?

Ensure complex protobuf files can be parsed

@bitwalker could you review this?