Skip to content
This repository was archived by the owner on Jul 9, 2025. It is now read-only.

Comments

fix: fix issue rendering delete properties action#2460

Merged
cwhitten merged 4 commits intomicrosoft:masterfrom
a-b-r-o-w-n:fix/delete-properties
Apr 2, 2020
Merged

fix: fix issue rendering delete properties action#2460
cwhitten merged 4 commits intomicrosoft:masterfrom
a-b-r-o-w-n:fix/delete-properties

Conversation

@a-b-r-o-w-n
Copy link
Contributor

Description

This happened because I was recursing on object types with properties defined which collided with a schema property named properties. This is unnecessary now as I don't need to recurse the schema.

Task Item

fixes #2458

Screenshots

image

@github-actions
Copy link

github-actions bot commented Apr 2, 2020

Coverage Status

Coverage increased (+0.1%) to 39.663% when pulling 2c9bff9 on a-b-r-o-w-n:fix/delete-properties into be2d064 on microsoft:master.

@cwhitten cwhitten merged commit e14850e into microsoft:master Apr 2, 2020
@a-b-r-o-w-n a-b-r-o-w-n deleted the fix/delete-properties branch April 17, 2020 17:17
lei9444 pushed a commit to lei9444/BotFramework-Composer-1 that referenced this pull request Jun 15, 2021
* fix issues rendering delete properties actions

* no longer recurse on schema

Co-authored-by: Chris Whitten <christopher.whitten@microsoft.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants