search

FlutterFlow / flutterflow-documentation

20 stars 41 forks source link

Updated AdMob Integrations #187

Closed pinkeshmars closed 2 weeks ago

pinkeshmars commented 1 month ago

Description

Updated AdMob Integrations

Linear ticket and magic word Fixes DEVR-357 Fixes DEVR-630

Type of change

stackblitz[bot] commented 1 month ago

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

PoojaB26 commented 3 weeks ago

Hey @pinkeshmars it seems the diagrams have started using pointed arrows instead of bullet arrows, can you please fix the arrows and to maintain consistency, not use pointed arrows anywhere (other than very special circumstances like flowcharts). I see this being used more lately, please go to Figma and check your last few diagrams and fix them and update the files in this PR (even if its not admob related, for example for new feature release docs)

pinkeshmars commented 2 weeks ago

Need few fixes.

Also, I know we mentioned about fixing the looping feature of Arcades? Like stopping the Arcade at the end of the slides and let users restart it. I see all the arcades here are looped as soon as the last slide is clicked. If we found any solution, can we apply them here?

Yes, we initially enabled the stepper to help users track their progress, so if they start from the first step, they’ll have a clear indication of where they are. However, it’s not a perfect solution. After exploring more options in Arcade, I found that we can resolve this by setting the hotspot's 'Destination' to 'This Step'. All arcades on this page are fixed now.

Please note that this issue only occurs when the last step contains a hotspot. We’ll need to apply this solution for all the arcades. We can discuss further how we should approach it.

Hey @pinkeshmars it seems the diagrams have started using pointed arrows instead of bullet arrows, can you please fix the arrows and to maintain consistency, not use pointed arrows anywhere (other than very special circumstances like flowcharts). I see this being used more lately, please go to Figma and check your last few diagrams and fix them and update the files in this PR (even if its not admob related, for example for new feature release docs)

Sure, other images are for localization, which is WIP, and streaming Supabase query, which is not merged yet, so I shall update in that PR only.

pinkeshmars commented 2 weeks ago

@PoojaB26 this is ready to review again.