[DA] Implement auto refresh limit - Githubissues

Econify / ad.js

🚀 Supercharge your Ads.

https://adjs.dev

18 stars 4 forks source link

[DA] Implement auto refresh limit #128

Closed alcocerd closed 3 years ago

alcocerd commented 3 years ago

Description

This change simply adds logic to support the refreshLimit described in our docs https://adjs.dev/#/refresh-plugin?id=options

PR Requirements

Before requesting review this criteria must be met: Overall test coverage >= current master?

[ ] Yes
[x] N.A. (note, I don't see any tests for the AutoRefresh plugin.

Documentation included (if any behavioral changes)?

[x] Yes (this was already included in our docs before this PR https://adjs.dev/#/refresh-plugin?id=options)
[ ] N.A.

Backwards compatibility (if breaking change)?

[ ] Yes
[x] N.A.

Release

Will this pr trigger a release i.e. version in package.json has been bumped?

[x] Yes
[ ] No

Screenshots

If U.I. component, include screenshots or video screen capture showing behavior and responsive styling below.

MatthewCMoya commented 3 years ago

There are not explicit tests for this file -- they are however being tested by extension via ad.test.ts