Skip to content

Fixed missing enum extend. #137#138

Merged
alaa-eddine merged 2 commits intoQuantForgeOrg:devfrom
dcaoyuan:fix-missing-extend
Mar 3, 2026
Merged

Fixed missing enum extend. #137#138
alaa-eddine merged 2 commits intoQuantForgeOrg:devfrom
dcaoyuan:fix-missing-extend

Conversation

@dcaoyuan
Copy link
Contributor

@dcaoyuan dcaoyuan commented Mar 3, 2026

Added enum extend.

@alaa-eddine alaa-eddine changed the base branch from main to dev March 3, 2026 23:52
@alaa-eddine alaa-eddine merged commit f553a9e into QuantForgeOrg:dev Mar 3, 2026
1 check passed
@github-actions github-actions bot locked and limited conversation to collaborators Mar 3, 2026
@alaa-eddine
Copy link
Collaborator

Merged, please note that I will change it to this, in order to match Pine Script internal representation

export enum extend {
    left = 'l',
    right = 'r',
    both = 'b',
    none = 'n',
}

there are also some other impacted types like belowbar, abovebar becoming BelowBar, AboveBar
and arrowup, arrowdown becoming shape_arrow_up, shape_arrow_down ...etc

I changed many Type enums values to match internal tradingview representation

In tradingview you can get the actual representations by using a code like this :

log.info('{0} {1} {2} {3} {4} {5}', str.tostring(location.abovebar), str.tostring(location.belowbar), str.tostring(location.top), str.tostring(location.bottom), str.tostring(location.absolute))

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