Skip to content

Paywalls: using PaywallData and setting up basic template loading#2781

Merged
NachoSoto merged 1 commit into
paywallsfrom
paywalls-using-config
Jul 13, 2023
Merged

Paywalls: using PaywallData and setting up basic template loading#2781
NachoSoto merged 1 commit into
paywallsfrom
paywalls-using-config

Conversation

@NachoSoto

@NachoSoto NachoSoto commented Jul 10, 2023

Copy link
Copy Markdown
Contributor

This adds a new TemplateViewType that different templates can conform to.
PaywallView now takes PaywallData and renders the paywall depending on its template.

@NachoSoto NachoSoto requested a review from a team July 10, 2023 23:20
@NachoSoto NachoSoto force-pushed the paywalls-using-config branch from e6062df to 938a24f Compare July 10, 2023 23:23
@NachoSoto NachoSoto force-pushed the paywalls-using-config branch from 938a24f to 576c60a Compare July 10, 2023 23:29
@NachoSoto NachoSoto force-pushed the paywall-config branch 2 times, most recently from 06c9e6f to 55ff780 Compare July 11, 2023 19:27
@NachoSoto NachoSoto force-pushed the paywalls-using-config branch 2 times, most recently from 46272c3 to c1a8f9e Compare July 11, 2023 20:03
@NachoSoto NachoSoto changed the title [WIP] Paywalls: using PaywallData and setting up basic template loading Paywalls: using PaywallData and setting up basic template loading Jul 11, 2023
@NachoSoto NachoSoto marked this pull request as ready for review July 11, 2023 20:04
@NachoSoto NachoSoto force-pushed the paywalls-using-config branch from c1a8f9e to 4291164 Compare July 11, 2023 20:07

@joshdholtz joshdholtz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM 💪

@NachoSoto NachoSoto force-pushed the paywalls-using-config branch from 4291164 to 7809cd7 Compare July 12, 2023 13:08
@NachoSoto NachoSoto force-pushed the paywall-config branch 11 times, most recently from 29de25c to 2f5f5bc Compare July 12, 2023 22:29
NachoSoto added a commit that referenced this pull request Jul 13, 2023
…#2781)

This adds a new `TemplateViewType` that different templates can conform
to.
`PaywallView` now takes `PaywallData` and renders the paywall depending
on its template.
NachoSoto added a commit that referenced this pull request Jul 13, 2023
…#2781)

This adds a new `TemplateViewType` that different templates can conform
to.
`PaywallView` now takes `PaywallData` and renders the paywall depending
on its template.
NachoSoto added a commit that referenced this pull request Jul 13, 2023
…#2781)

This adds a new `TemplateViewType` that different templates can conform
to.
`PaywallView` now takes `PaywallData` and renders the paywall depending
on its template.
NachoSoto added a commit that referenced this pull request Jul 13, 2023
…#2781)

This adds a new `TemplateViewType` that different templates can conform
to.
`PaywallView` now takes `PaywallData` and renders the paywall depending
on its template.
NachoSoto added a commit that referenced this pull request Jul 17, 2023
…#2781)

This adds a new `TemplateViewType` that different templates can conform
to.
`PaywallView` now takes `PaywallData` and renders the paywall depending
on its template.
NachoSoto added a commit that referenced this pull request Jul 18, 2023
…#2781)

This adds a new `TemplateViewType` that different templates can conform
to.
`PaywallView` now takes `PaywallData` and renders the paywall depending
on its template.
NachoSoto added a commit that referenced this pull request Jul 18, 2023
…#2781)

This adds a new `TemplateViewType` that different templates can conform
to.
`PaywallView` now takes `PaywallData` and renders the paywall depending
on its template.
NachoSoto added a commit that referenced this pull request Jul 20, 2023
…#2781)

This adds a new `TemplateViewType` that different templates can conform
to.
`PaywallView` now takes `PaywallData` and renders the paywall depending
on its template.
NachoSoto added a commit that referenced this pull request Jul 23, 2023
…#2781)

This adds a new `TemplateViewType` that different templates can conform
to.
`PaywallView` now takes `PaywallData` and renders the paywall depending
on its template.
NachoSoto added a commit that referenced this pull request Jul 24, 2023
…#2781)

This adds a new `TemplateViewType` that different templates can conform
to.
`PaywallView` now takes `PaywallData` and renders the paywall depending
on its template.
NachoSoto added a commit that referenced this pull request Jul 24, 2023
…#2781)

This adds a new `TemplateViewType` that different templates can conform
to.
`PaywallView` now takes `PaywallData` and renders the paywall depending
on its template.
NachoSoto added a commit that referenced this pull request Jul 25, 2023
…#2781)

This adds a new `TemplateViewType` that different templates can conform
to.
`PaywallView` now takes `PaywallData` and renders the paywall depending
on its template.
NachoSoto added a commit that referenced this pull request Jul 26, 2023
…#2781)

This adds a new `TemplateViewType` that different templates can conform
to.
`PaywallView` now takes `PaywallData` and renders the paywall depending
on its template.
NachoSoto added a commit that referenced this pull request Jul 27, 2023
…#2781)

This adds a new `TemplateViewType` that different templates can conform
to.
`PaywallView` now takes `PaywallData` and renders the paywall depending
on its template.
NachoSoto added a commit that referenced this pull request Jul 31, 2023
…#2781)

This adds a new `TemplateViewType` that different templates can conform
to.
`PaywallView` now takes `PaywallData` and renders the paywall depending
on its template.
NachoSoto added a commit that referenced this pull request Aug 3, 2023
…#2781)

This adds a new `TemplateViewType` that different templates can conform
to.
`PaywallView` now takes `PaywallData` and renders the paywall depending
on its template.
NachoSoto added a commit that referenced this pull request Aug 7, 2023
…#2781)

This adds a new `TemplateViewType` that different templates can conform
to.
`PaywallView` now takes `PaywallData` and renders the paywall depending
on its template.
NachoSoto added a commit that referenced this pull request Aug 9, 2023
…#2781)

This adds a new `TemplateViewType` that different templates can conform
to.
`PaywallView` now takes `PaywallData` and renders the paywall depending
on its template.
NachoSoto added a commit that referenced this pull request Aug 11, 2023
…#2781)

This adds a new `TemplateViewType` that different templates can conform
to.
`PaywallView` now takes `PaywallData` and renders the paywall depending
on its template.
NachoSoto added a commit that referenced this pull request Aug 14, 2023
…#2781)

This adds a new `TemplateViewType` that different templates can conform
to.
`PaywallView` now takes `PaywallData` and renders the paywall depending
on its template.
NachoSoto added a commit that referenced this pull request Aug 17, 2023
…#2781)

This adds a new `TemplateViewType` that different templates can conform
to.
`PaywallView` now takes `PaywallData` and renders the paywall depending
on its template.
NachoSoto added a commit that referenced this pull request Aug 24, 2023
…#2781)

This adds a new `TemplateViewType` that different templates can conform
to.
`PaywallView` now takes `PaywallData` and renders the paywall depending
on its template.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants