Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

General: Use query functions in rest api calls #3457

Merged

Conversation

iLLiCiTiT
Copy link
Member

Brief description

Rest api calls in avalon module use query functions instead of AvalonMongoDB.

Additional info

Not sure if are the rest points used anywhere? Maybe we could removed them.

Testing notes:

It should be possible to receive project documents using rest api points. Try access http://localhost:8079/avalon/projects when tray is running.

@ynbot
Copy link
Contributor

ynbot commented Jul 1, 2022

@iLLiCiTiT iLLiCiTiT requested review from 64qam and kalisp July 1, 2022 07:21
@iLLiCiTiT iLLiCiTiT self-assigned this Jul 1, 2022
@iLLiCiTiT iLLiCiTiT added the type: refactor Structural changes not affecting functionality label Jul 1, 2022
@iLLiCiTiT iLLiCiTiT merged commit a754f1a into develop Jul 1, 2022
@iLLiCiTiT iLLiCiTiT deleted the feature/OP-3410_Use-query-functions-in-avalon-module branch July 1, 2022 09:55
Comment on lines +25 to +28
output = [
project_doc
for project_doc in get_projects()
]
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is there a reason this isn't just output = list(get_projects())?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: refactor Structural changes not affecting functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants