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

Fmd 984 replace owner refs from app #1045

Merged
merged 2 commits into from
Nov 19, 2024

Conversation

hjribeiro-moj
Copy link
Contributor

@hjribeiro-moj hjribeiro-moj commented Nov 13, 2024

  • replaced data_owner references from app code
  • kept existing references on datahub lib

keeping PR short to minimize reference problems

it works now in dev:

Uploading image.png…

 - replaced `data_owner` with `data_custodian` in the codebase
 - did not replace any reference to data owner on datahub lib

Signed-off-by: Helder Ribeiro <[email protected]>
Signed-off-by: Helder Ribeiro <[email protected]>
Copy link
Contributor

@murdo-moj murdo-moj left a comment

Choose a reason for hiding this comment

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

LGTM

@hjribeiro-moj hjribeiro-moj merged commit bf78d63 into main Nov 19, 2024
18 checks passed
@hjribeiro-moj hjribeiro-moj deleted the fmd-984-replace-owner-refs-from-app branch November 19, 2024 16:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants