Conversation
|
Preview this PR here: https://dev-docs.revenuecat.com/pr-1313/ |
sidebars.ts
Outdated
| link: "dashboard-and-metrics/charts/subscriptions", | ||
| }, | ||
| }), | ||
| SubCategory({ |
There was a problem hiding this comment.
Until we do a public release we should skip adding these to the sidebar and just make them reachable via the charts links in the Dashboard that only appear when the relevant feature flags are active. You can leave a PR open and ready with the change in this file for when the time comes.
| @@ -0,0 +1,50 @@ | |||
| --- | |||
| title: Ad Impressions Chart | |||
There was a problem hiding this comment.
Note
This applies to all the charts
We removed the Ad prefix prefix for all ads charts in the Dashboard since it felt redundant. This title is also used in the sidebar where the redundancy is obvious. For consistency I would suggest removing it here as well. Not a strong opinion by any means.
sidebars.ts
Outdated
| ], | ||
| index: { | ||
| title: "Ads", | ||
| link: "dashboard-and-metrics/charts/ads", |
There was a problem hiding this comment.
I think this documentation page does not exist yet?
…f the language, and add in some details about Ad Format
|
FYI @peterporfy added a little about ad format now that you've added support |
2 participants
Motivation / Description / Changes introduced
This PR adds all the necessary documentation supporting the new ads charts we're introducing as part of the ads tracking feature.