Skip to content

Conversation

@iLLiCiTiT
Copy link
Member

@iLLiCiTiT iLLiCiTiT commented Mar 17, 2025

Changelog Description

Server added option to filter users by email in GraphQl.

Additional review information

It is not yet available in any server release, so we should wait with merging. ValueError is raised if is connected to older server version.

Testing notes:

  1. Run current develop of server backend (or 1.7.4 if already released).
  2. Use get_users with emails filter.

@iLLiCiTiT iLLiCiTiT self-assigned this Mar 17, 2025
@iLLiCiTiT iLLiCiTiT added the type: enhancement New feature or request label Mar 17, 2025
@iLLiCiTiT iLLiCiTiT requested review from kalisp and martastain March 17, 2025 13:02
Copy link
Member

@kalisp kalisp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Isn't emails key argument missing in ayon_api/_api.py.get_users? This wasn't working for me until I added it there and propagated inside that function.

kalisp

This comment was marked as duplicate.

@iLLiCiTiT iLLiCiTiT requested a review from kalisp April 8, 2025 08:50
Copy link
Member

@kalisp kalisp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

emails keyword works in get_users

@iLLiCiTiT iLLiCiTiT merged commit 9532111 into develop Apr 14, 2025
2 checks passed
@iLLiCiTiT iLLiCiTiT deleted the enhancement/add-emails-filtering branch April 14, 2025 12:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants