Skip to content

Conversation

Rodriq
Copy link
Contributor

@Rodriq Rodriq commented Aug 7, 2024

Description

notice that some actions are not available in the action block

Type of change

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

…ction Block

notice that some actions are not available in the action block
Copy link

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

@PoojaB26 PoojaB26 self-requested a review August 7, 2024 11:46
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.

Hi @Rodriq, thanks for providing a PR. However, we use a different syntax for callouts in this project. Its called Admonitions, and please see the syntax for adding a notes admonition. Here is the contribution guidelines link (also linked in README now)

Used correct style and added some example actions not available for use in an Action block
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.

Suggested some language and title changes, please check and Accept Changes if that works with you!

Please accept the suggestions in a single commit instead of 3 different commits.
Check pt 3.2 in the Github PR guide.

@Rodriq
Copy link
Contributor Author

Rodriq commented Aug 16, 2024

@PoojaB26 I could open a new PR if needed.
I accepted your suggestions one after the other.

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.

No worries, you can take care of it in a future PR so all such suggested changes can be part of one single commit. I'm approving it now and merging, thanks!!!

@PoojaB26 PoojaB26 merged commit 9bcce5f into FlutterFlow:main Aug 16, 2024
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