Skip to content

Commit

Permalink
Fix confidential and product category fields not being set when creat…
Browse files Browse the repository at this point in the history
…ing enteprrise feature (#4650)

The field were not properly being set because they were not passed to the FeatureEntry constructor
  • Loading branch information
yanndago authored Dec 20, 2024
1 parent 3fa4464 commit e495689
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
1 change: 0 additions & 1 deletion client-src/elements/form-field-specs.ts
Original file line number Diff line number Diff line change
Expand Up @@ -498,7 +498,6 @@ export const ALL_FIELDS: Record<string, Field> = {
choices: ENTERPRISE_PRODUCT_CATEGORY,
label: 'Enterprise product category',
help_text: html` Select the appropriate category.`,
check: (_value, getFieldValue) => checkFeatureNameAndType(getFieldValue),
},

feature_type: {
Expand Down
2 changes: 2 additions & 0 deletions pages/guide.py
Original file line number Diff line number Diff line change
Expand Up @@ -170,6 +170,8 @@ def process_post_data(self, **kwargs):
screenshot_links=self.parse_links('screenshot_links'),
first_enterprise_notification_milestone=enterprise_notification_milestone,
blink_components=[settings.DEFAULT_ENTERPRISE_COMPONENT],
confidential=self.form.get('confidential') == 'on',
enterprise_product_category=int(self.form.get('enterprise_product_category', '0')),
tag_review_status=core_enums.REVIEW_NA)
key: ndb.Key = feature_entry.put()
search_fulltext.index_feature(feature_entry)
Expand Down

0 comments on commit e495689

Please sign in to comment.