Skip to content

Conversation

@ZSnake
Copy link
Contributor

@ZSnake ZSnake commented Oct 1, 2024

Add an optional parameter to sections configuration for a custom renderItem function

@ZSnake ZSnake requested a review from a team as a code owner October 1, 2024 17:31
Copy link
Contributor

@esezen esezen left a comment

Choose a reason for hiding this comment

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

Thanks for working on this! The functionality works great, I left some comments about the docs

@@ -1,3 +1,4 @@
import React from 'react';
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we update the functionStrings object under utils to make sure the actual code shows up here (instead of render_CODE)?
Screenshot 2024-10-08 at 5 35 48 PM

Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Contributor

Choose a reason for hiding this comment

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

Screenshot 2024-10-09 at 3 09 08 PM

mocca102 and others added 3 commits October 9, 2024 13:40
Co-authored-by: Enes Kutay SEZEN <eneskutaysezen@gmail.com>
Co-authored-by: Enes Kutay SEZEN <eneskutaysezen@gmail.com>
@@ -1,3 +1,4 @@
import React from 'react';
Copy link
Contributor

Choose a reason for hiding this comment

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

import react statement required since there's JSX on the story file

@mocca102 mocca102 requested a review from esezen October 9, 2024 12:07
Copy link
Contributor

@esezen esezen left a comment

Choose a reason for hiding this comment

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

LGTM!

@mocca102 mocca102 merged commit 43746b6 into main Oct 15, 2024
10 checks passed
@mocca102 mocca102 deleted the ci-3883-os-ui-autocomplete-support-custom-pricing-for-product branch October 15, 2024 16:26
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.

4 participants