Skip to content

Conversation

@Marc-Roig
Copy link
Contributor

@Marc-Roig Marc-Roig commented Feb 7, 2023

What does it do?

Mentions connect attribute can not be used when managing relations of media attributes.

Why is it needed?

Some users have been complaining about this behaviour. We forgot to mention this is not supported 😄

Related issue(s)/PR(s)

strapi/strapi#15601

@vercel
Copy link

vercel bot commented Feb 7, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated
documentation ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Feb 7, 2023 at 1:53PM (UTC)
documentation-docu-mvp ❌ Failed (Inspect) Feb 7, 2023 at 1:53PM (UTC)


`connect` can be used in combination with [`disconnect`](#disconnect).

:::warning
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
:::warning
:::caution

Warning is the highest-level alert. It should only be used for cases that break the app or delete data :)

Copy link
Collaborator

@pwizla pwizla left a comment

Choose a reason for hiding this comment

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

Can you please just downgrade the callout to caution? Other than that, LGTM, thanks!

@pwizla pwizla added this to the 2023-6: 4.6.1 milestone Feb 7, 2023
@Marc-Roig
Copy link
Contributor Author

done! :)

@pwizla pwizla merged commit 18dc7a2 into main Feb 7, 2023
@pwizla pwizla deleted the dev/connect-relations-not-working-with-media branch February 7, 2023 14:07
@pwizla pwizla added internal PRs created by the Strapi core team replicate with docu labels Feb 7, 2023
gpene added a commit that referenced this pull request Feb 8, 2023
* user-permissions plugin: document how to add Patreon as an Oauth provider

* add server webhooks populateRelations config variable

* Update docs/developer-docs/latest/setup-deployment-guides/configurations/required/server.md

Co-authored-by: Pierre Wizla <pwizla@users.noreply.github.com>

* Auto stash before merge of "dev/webhook-populated-relations" and "origin/dev/webhook-populated-relations"

* add 4.6.1 migration guide

* Update docs/developer-docs/latest/setup-deployment-guides/configurations/required/server.md

Co-authored-by: Pierre Noël <petersg83@users.noreply.github.com>

* Update docs/developer-docs/latest/update-migration-guides/migration-guides/v4/migration-guide-4.5.1-to-4.6.1.md

Co-authored-by: Pierre Noël <petersg83@users.noreply.github.com>

* Update docs/developer-docs/latest/update-migration-guides/migration-guides/v4/migration-guide-4.5.1-to-4.6.1.md

Co-authored-by: Pierre Noël <petersg83@users.noreply.github.com>

* Update docs/developer-docs/latest/update-migration-guides/migration-guides/v4/migration-guide-4.5.1-to-4.6.1.md

Co-authored-by: Pierre Noël <petersg83@users.noreply.github.com>

* improve explanation of webhooks populate relations usage

* Update filtering-locale-publication.md (#1441)

for this pull
strapi/strapi#15046

* Fix deployment instructions for Azure CLI

# Change 1: Update Node runtime argument of ```az webapp create```

Running the docs as-is produces an error:
```... Linux Runtime 'node|10.14' is not supported.Run 'az webapp list-runtimes --os-type linux' to cross check```

The correct parameter today is

```bash
--runtime "node|14-lts"
```

# Change 2: Update IP address block arguments of ```az mysql server firewall-rule create```

Running the docs as-is produces an error:
```... the following arguments are required: --start-ip-address, --end-ip-address```

Based on [the most recent docs](https://learn.microsoft.com/en-us/cli/azure/mysql/server/firewall-rule?view=azure-cli-latest), the correct parameters to define the allowed IP blocks are:

```bash
--start-ip-address 0.0.0.0
--end-ip-address 0.0.0.0
```

* Change Node 14 to Node 16

https://azure.microsoft.com/en-us/updates/community-support-for-node-14-lts-is-ending-on-30-april-2023/

* Update docs/developer-docs/latest/plugins/users-permissions.md

* Update docs/developer-docs/latest/plugins/users-permissions.md

* Fix broken links reported in issue #1423 (#1484)

* Fix broken links reported in issue #1423

* Fix BE custom. > middlewares link

* Update quick-start.md (#1478)

* Move sizeLimit one level above in upload provider (#1133)

* move sizeLimit one level above in upload provider

* add sizeLimit deprecation warning

* minor change

* minor change

* checkFileSize upload provider

* Update docs/developer-docs/latest/update-migration-guides/migration-guides/v4/migration-guide-4.3.x-to-4.4.x.md

Co-authored-by: Pierre Wizla <pwizla@users.noreply.github.com>

* Update docs/developer-docs/latest/update-migration-guides/migration-guides/v4/migration-guide-4.3.x-to-4.4.x.md

Co-authored-by: Pierre Wizla <pwizla@users.noreply.github.com>

* Update docs/developer-docs/latest/update-migration-guides/migration-guides/v4/migration-guide-4.3.x-to-4.4.x.md

Co-authored-by: Pierre Wizla <pwizla@users.noreply.github.com>

* Update docs/developer-docs/latest/update-migration-guides/migration-guides/v4/migration-guide-4.3.x-to-4.4.x.md

Co-authored-by: Pierre Wizla <pwizla@users.noreply.github.com>

* Update docs/developer-docs/latest/update-migration-guides/migration-guides/v4/migration-guide-4.3.x-to-4.4.x.md

Co-authored-by: Pierre Wizla <pwizla@users.noreply.github.com>

* Update docs/developer-docs/latest/update-migration-guides/migration-guides/v4/migration-guide-4.3.x-to-4.4.x.md

Co-authored-by: Pierre Wizla <pwizla@users.noreply.github.com>

* Update docs/developer-docs/latest/update-migration-guides/migration-guides/v4/migration-guide-4.3.x-to-4.4.x.md

Co-authored-by: Pierre Wizla <pwizla@users.noreply.github.com>

* move migration steps to 4.5.1 to 4.6.1

---------

Co-authored-by: Pierre Wizla <pwizla@users.noreply.github.com>

* update localhost link

* Connect can not be used for media attributes (#1490)

* connect can not be used for meida attributes

* change warning to caution

---------

Co-authored-by: Tom Shortall <tomshortall@gmail.com>
Co-authored-by: Marc-Roig <marc12info@gmail.com>
Co-authored-by: Pierre Wizla <pwizla@users.noreply.github.com>
Co-authored-by: Pierre Noël <petersg83@users.noreply.github.com>
Co-authored-by: Jorge Martin <53431308+HastaCs@users.noreply.github.com>
Co-authored-by: Antti <antti.viitala@icloud.com>
Co-authored-by: Shaun Brown <97027841+StrapiShaun@users.noreply.github.com>
Co-authored-by: Pierre Wizla <pwizla+github@gmail.com>
Co-authored-by: katekosm <71697829+katekosm@users.noreply.github.com>
pwizla added a commit that referenced this pull request Feb 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal PRs created by the Strapi core team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants