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.

Sign up for GitHub

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

Jump to bottom

Chore: Move Harmony client code #568

Merged
merged 16 commits into from
Jun 5, 2024
Merged

Chore: Move Harmony client code #568

merged 16 commits into from
Jun 5, 2024

Conversation

iLLiCiTiT
Copy link
Member

Changelog Description

Moved harmony client code next to server codebase.

Additional info

Change file structure of stdout_broker tool. Class StdOutBroken moved to broker.py and added it to __init__.py of the tool.

Testing notes:

  1. Create package of harmony and core.
  2. Upload both to server.
  3. Make sure AYON launcher is using client code from this PR (e.g using dev paths).
  4. Harmony should work as before.

@iLLiCiTiT iLLiCiTiT requested a review from kalisp May 30, 2024 11:39
@ynbot
Copy link
Contributor

ynbot commented May 30, 2024

Task linked: AY-4914 Move Harmony client code

@ynbot ynbot added type: enhancement Improvement of existing functionality or minor addition size/S host: Harmony labels May 30, 2024
@iLLiCiTiT iLLiCiTiT changed the title Chore: Move harmony client code Chore: Move Harmony client code Jun 3, 2024
@kalisp kalisp assigned iLLiCiTiT and unassigned kalisp Jun 5, 2024
@iLLiCiTiT iLLiCiTiT merged commit f079455 into develop Jun 5, 2024
1 check passed
@iLLiCiTiT iLLiCiTiT deleted the enhancement/AY-4914_Move-Harmony-client-code branch June 5, 2024 11:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kalisp kalisp kalisp approved these changes

Assignees

@iLLiCiTiT iLLiCiTiT

Labels
size/S type: enhancement Improvement of existing functionality or minor addition
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@iLLiCiTiT @ynbot @kalisp