Skip to content

Conversation

@bmtcril
Copy link
Contributor

@bmtcril bmtcril commented Sep 11, 2024

Adds tag, taxonomy, and object tag test data population to the script.

@bmtcril bmtcril force-pushed the bmtcril/add_tags_and_taxonomies branch 2 times, most recently from b66cf82 to aa8217a Compare September 18, 2024 19:42
@bmtcril bmtcril marked this pull request as ready for review September 18, 2024 19:43
Copy link
Contributor

@saraburns1 saraburns1 left a comment

Choose a reason for hiding this comment

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

ran locally - lgtm

@bmtcril
Copy link
Contributor Author

bmtcril commented Sep 23, 2024

@saraburns1 @Ian2012 If anyone wants to take another review of the last commit it's there, otherwise I'll merge this in a couple of hours.

The integer parent id was hard coded to 0 previously, but it seems that
we're using it, so now it is included.
Fix typo, default tag parent_id to 0 instead of null.
@bmtcril bmtcril force-pushed the bmtcril/add_tags_and_taxonomies branch from 05b95f9 to 8e7c8e3 Compare September 23, 2024 17:09
@bmtcril bmtcril merged commit b2c912d into main Sep 23, 2024
@bmtcril bmtcril deleted the bmtcril/add_tags_and_taxonomies branch September 23, 2024 17:13
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.

3 participants