Skip to content
This repository has been archived by the owner on Sep 20, 2024. It is now read-only.

General: Mongo core connection moved to client #3531

Merged
merged 8 commits into from
Jul 26, 2022

Conversation

iLLiCiTiT
Copy link
Member

@iLLiCiTiT iLLiCiTiT commented Jul 18, 2022

Brief description

Moved core of Mongo connection OpenPypeMongoConnection from openpype.lib into openpype.client. This change makes openpype.client independent from openpype lib functions and can be considered as self contained.

Description

Changed parts of code using OpenPypeMongoConnection to use new source and marked lib functions as deprecated. That is unfortunatelly not possible for classes located there too.

Testing notes:

  • Logging to mongo should work (can be validated using LogViewer in tray)
  • Webpublisher should work
  • Maya shader publishing can be possible (not sure if is used?)
  • OpenPype MongoURL can be changed in local settings UI
  • Remote (automated) publishing can get and update jobs to do
  • Ftrack event server (service) can process ftrack events
  • Integration to slack should work
  • Project Settings can be loaded and saved

@ynbot
Copy link
Contributor

ynbot commented Jul 18, 2022

Task linked: OP-3595 Mongo core connection in client

@iLLiCiTiT iLLiCiTiT added the type: refactor Structural changes not affecting functionality label Jul 18, 2022
@iLLiCiTiT iLLiCiTiT self-assigned this Jul 18, 2022
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.

Tested webpublishing, automatic testing, works.

@iLLiCiTiT iLLiCiTiT merged commit c05a510 into develop Jul 26, 2022
@iLLiCiTiT iLLiCiTiT deleted the feature/OP-3595_Mongo-core-connection-in-client branch July 26, 2022 11:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
type: refactor Structural changes not affecting functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants