Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[next] chore(docs): update documentation #6482

Open
wants to merge 10 commits into
base: next
Choose a base branch
from
Open

Conversation

backportbot[bot]
Copy link

@backportbot backportbot bot commented Feb 3, 2025

Backport of #6460

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Remove all the empty commits

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

@backportbot backportbot bot requested review from skjnldsv, susnux and ShGKme February 3, 2025 07:32
@backportbot backportbot bot added 3. to review Waiting for reviews feature: documentation Related to the documentation technical debt labels Feb 3, 2025
Signed-off-by: Grigorii K. Shartsev <[email protected]>

chore(docs): import instructions

Signed-off-by: Grigorii K. Shartsev <[email protected]>
Signed-off-by: Grigorii K. Shartsev <[email protected]>
Signed-off-by: Grigorii K. Shartsev <[email protected]>
Signed-off-by: Grigorii K. Shartsev <[email protected]>
Signed-off-by: Grigorii K. Shartsev <[email protected]>
@ShGKme ShGKme force-pushed the backport/6460/next branch from b096157 to 2e761e6 Compare February 3, 2025 07:44
@ShGKme ShGKme marked this pull request as ready for review February 3, 2025 07:44
```

## Recommendations
### (Deprecated) Registering all components
Copy link
Contributor

Choose a reason for hiding this comment

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

I think this section can go now, no?

Copy link
Contributor

Choose a reason for hiding this comment

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

Hmm, I though, I checked every commit on backport 🙈


You can also import individual module without bundling the full library.
// (Deprecated) Old import path
Copy link
Contributor

Choose a reason for hiding this comment

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

This can also be removed for next.

import NcButton from '@nextcloud/vue/components/NcButton'
import { useHotKey } from '@nextcloud/vue/composables/useHotKey'

// ⚠️ (REMOVED) In v8 the following import path could be used. It was deprecated and removed in v9.
Copy link
Contributor

Choose a reason for hiding this comment

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

And this as well 🙂

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews feature: documentation Related to the documentation technical debt
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants