cli: Add collection arg

[A6GibKm]

No description provided.

[bilelmoussaoui]

Can you rebase this one please?

[A6GibKm]

Can you rebase this one please?

Rebased, but I think we should use Login. Note too that this string is translatable in other projects.

[bilelmoussaoui]

Can you rebase this one please?

Rebased, but I think we should use Login. Note too that this string is translatable in other projects.

Hmm, that makes providing both the helper methods & constants weird. As the CLI tool could become translatable, not the oo7 library itself...

[A6GibKm]

Can you rebase this one please?

Rebased, but I think we should use Login. Note too that this string is translatable in other projects.

Hmm, that makes providing both the helper methods & constants weird. As the CLI tool could become translatable, not the oo7 library itself...

Sounds like a problem for tomorrow, but it makes sense to have it translatable everywhere.

[bilelmoussaoui]

Can you rebase this one please?

Rebased, but I think we should use Login. Note too that this string is translatable in other projects.

Hmm, that makes providing both the helper methods & constants weird. As the CLI tool could become translatable, not the oo7 library itself...

Sounds like a problem for tomorrow, but it makes sense to have it translatable everywhere.

I went ahead and add various helper APIs here that makes the whole thing nicer while allowing people to translate things if they want to.

[A6GibKm]

rebased, but please don't merge just yet, the arg is not properly working , e.g.

$ cargo run -- search --collection c a=b
    Finished `dev` profile [unoptimized + debuginfo] target(s) in 0.40s
     Running `/var/home/msandova/Projects/oo7/target/debug/oo7-cli search --collection c a=b`
error: unexpected argument '--collection' found

  tip: to pass '--collection' as a value, use '-- --collection'

Usage: oo7-cli search [OPTIONS] [ATTRIBUTES]...

For more information, try '--help'.

[bilelmoussaoui]

otherwise lgtm, once you solved the cli arg parser issue :)

[A6GibKm]

otherwise lgtm, once you solved the cli arg parser issue :)

Done, it was a matter of using global=true.