Skip to content

Conversation

pinkeshmars
Copy link
Collaborator

Description

Provide a brief overview of what this documentation update is about. Explain what sections or topics are being added or revised.

Linear ticket and magic word Fixes DEVR-539

Type of change

  • Typo fix
  • New feature
  • Enhancement to current docs
  • Removed outdated references
  • Update assets

@pinkeshmars pinkeshmars requested a review from PoojaB26 August 1, 2024 05:57
Copy link

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

Copy link
Collaborator

@PoojaB26 PoojaB26 left a comment

Choose a reason for hiding this comment

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

Please make the changes in the PR and then merge!

You should be creating two AdMob Apps to display advertisements in both Android and iOS apps.
:::

<div style={{
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why is this Arcade on "sharing project with user" here?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Fixed!

4. Find the **AdMob** **Android App Key** input box and enter the key.
5. Similarly, add the **AdMob** **iOS App Key** from the iOS AdMob app.

<div style={{
Copy link
Collaborator

Choose a reason for hiding this comment

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

Again, irrelevant arcade.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Fixed!

@@ -0,0 +1,481 @@
---
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please check the Arcades on the page. Seems to be wrong.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Fixed!

---
slug: general-settings
title: General Settings
tags: [project, assets, splash screen, launcher icon, adaptive icon]
Copy link
Collaborator

Choose a reason for hiding this comment

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

Tags should start with capital letters. And they should be relevant to multiple pages. Keywords can be any but tags need to be few.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Only Project and Assets should be there in tags.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Updated!

:::tip
After changing the package name, errors may appear on the toolbar due to invalidated Firebase config files. To resolve this, generate new config files by going to **Settings & Integrations > Project Setup > Firebase > Regenerate Config Files**.
:::
---
Copy link
Collaborator

Choose a reason for hiding this comment

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

We dont need any divider in this page. Please remove them throughout the page.

Copy link
Collaborator Author

@pinkeshmars pinkeshmars Aug 1, 2024

Choose a reason for hiding this comment

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

Removed

@@ -4,6 +4,7 @@ title: How to Create, Find, and Organize Projects
description: Learn how to create, find, and organize projects in FlutterFlow to streamline your app development process.
tags: [Project Management, FlutterFlow, Organization, Projects]
toc_max_heading_level: 5
sidebar_position: 3
Copy link
Collaborator

Choose a reason for hiding this comment

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

Create project should be just after what is a project. So this should be second in order.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Updated!

---
slug: project-setup
title: Project Setup
tags: [permissions, multiple languages, multi-platform, walkthrough]
Copy link
Collaborator

Choose a reason for hiding this comment

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

Same comment about tags starting with capital letter. And I know we use ChatGPT for it, but we should carefully filter the tags in each page. Each time we add a tag here, Docusaurus will create a new page called ../tags/permissions so all pages tagged permissions will appear there. So we need to make sure we dont create unnecessary tags. Please see all the pages in this PR and ensure we only create relevant tags that can be reused for diffrent pages (example "setup" can be reused between flutterflow setup, app setup, project setup)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes, got it!

@pinkeshmars pinkeshmars merged commit 4c1df9b into main Aug 1, 2024
@PoojaB26 PoojaB26 deleted the pinkesh/missing-items branch October 2, 2024 06:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants