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

issues/1441 : Bug fix for picklist Deployment #1442

Merged
merged 3 commits into from
Nov 24, 2023

Conversation

cjbradshaw
Copy link
Contributor

@cjbradshaw cjbradshaw commented Nov 23, 2023

fixes the "Cannot read properties of undefined (reading 'Metadata')" included some trace logging for troubleshooting

Summary generated by Reviewpad on 23 Nov 23 10:52 UTC

This pull request is a bug fix for the issues/1441. It fixes the bug that causes "Cannot read properties of undefined (reading 'Metadata')" error. The patch includes some trace logging for troubleshooting.

Checklist

All items have to be completed before a PR is merged

  • Adhere to Contribution Guidelines
  • Updates to Decision Records considered?
  • Updates to documentation at DX@Scale Guide considered?
  • Tested changes?
  • Unit Tests new and existing passing locally?

fixes the Cannot read properties of undefined (reading 'Metadata')
included some trace logging for troubleshooting
@reviewpad reviewpad bot requested a review from Rocko1204 November 23, 2023 10:52
@reviewpad reviewpad bot added small Pull request is small waiting-for-review labels Nov 23, 2023
Copy link

reviewpad bot commented Nov 23, 2023

Reviewpad Report

‼️ Errors

  • Unconventional commit detected: 'issues/1441 : Bug fix for picklist Deployment
    fixes the Cannot read properties of undefined (reading 'Metadata')
    included some trace logging for troubleshooting' (16d400c)
  • Unconventional commit detected: 'Merge branch 'main' into issues/1441' (ef66d81)
  • Unconventional title detected: 'issues/1441 : Bug fix for picklist Deployment' illegal 'i' character in commit message type: col=00

⚠️ Warnings

  • Please link an issue to the pull request

Copy link
Contributor

@azlam-abdulsalam azlam-abdulsalam left a comment

Choose a reason for hiding this comment

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

Thanks @cjbradshaw

@azlam-abdulsalam azlam-abdulsalam merged commit 65e36ee into dxatscale:main Nov 24, 2023
3 of 4 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
small Pull request is small waiting-for-review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants