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

Fix copy-usages not working #32

Merged
merged 1 commit into from
Jun 4, 2024
Merged

Conversation

drscholly
Copy link
Contributor

@drscholly drscholly commented Apr 10, 2024

Checklist

  • I have read the Contribution Guide ;
  • I have read the Contributor License Agreement (CLA) and understand that the submission of this PR will constitute an electronic signature of my agreement of the terms and conditions of DataGalaxy's CLA ;
  • There is an approved issue describing the change when contributing a new feature ;
  • I have added tests that prove my fix is effective or that my feature works ;
  • I have added necessary documentation (if appropriate) ;

Short description of what this resolves:

The app tried to access the children property without checking if it exists, resulting in an error when it is the case. Added a security check.

Must check if there is a children before trying
to edit it
@drscholly drscholly requested a review from a team as a code owner April 10, 2024 10:13
@sailro sailro merged commit dcc54e8 into datagalaxy-lab:main Jun 4, 2024
4 checks passed
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.

None yet

2 participants