Skip to content

feat: Add is_extended_promotional column to all components#113

Open
MuhamadJuwandi wants to merge 1 commit intotscircuit:mainfrom
MuhamadJuwandi:feature/extended-promotional
Open

feat: Add is_extended_promotional column to all components#113
MuhamadJuwandi wants to merge 1 commit intotscircuit:mainfrom
MuhamadJuwandi:feature/extended-promotional

Conversation

@MuhamadJuwandi
Copy link

Closes #92. This PR adds the 'is_extended_promotional' boolean column to BaseComponent and all derived tables. It includes logic to extract this flag from the component's 'extra' attributes.

Added is_extended_promotional boolean column to BaseComponent and all derived tables. Created logic to extract this flag from component 'extra' attributes.
Copy link
Member

@techmannih techmannih left a comment

Choose a reason for hiding this comment

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

Add screenshot after running it locally.

@techmannih
Copy link
Member

Join our discord for fast review and discussion. Feel Free to contribute.

https://discord.gg/NvmyMWczE

@MuhamadJuwandi
Copy link
Author

MuhamadJuwandi commented Dec 28, 2025

Screenshot 2025-12-28 154334 Added verification script scripts/verify-extended-promo.ts and confirmed logic works locally. Screenshot attached.

@MuhamadJuwandi
Copy link
Author

/claim #92

Copy link

@rushabhcodes rushabhcodes left a comment

Choose a reason for hiding this comment

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

I believe this issue is already solved #99

@MuhamadJuwandi
Copy link
Author

I've checked PR #99 , and it appears to only add the is_preferred column. While related, Issue #92 specifically requested the is_extended_promotional column to identify parts that act as "basic" for a limited time (Extended Promotional).

In this PR (#113), I've added the exact is_extended_promotional field as requested. Checking the logic, this field provides distinct data from is_preferred and is necessary to fully resolve the requirements of #92

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.

Add is_extended_promotional column to components (from data source)

3 participants