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

[VO-421] fix: Taking account of feedback following the migration of apps to 11 #835

Merged
merged 6 commits into from
Feb 29, 2024

Conversation

cballevre
Copy link
Contributor

No description provided.

@cballevre cballevre force-pushed the fix/v11-migration-returns branch from ebe0a4f to 1792480 Compare February 28, 2024 09:44
const appReducer = combineReducers(reducers)
```

If you did not provide a custom store to cozy-client, you may need to add the `store: true` option when setting up your CozyClient to ensure that it has a store. This was previously done as a side effect in cozy-bar v7.
Copy link
Contributor

Choose a reason for hiding this comment

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

on devrait mettre plus en avant ce cas, peut-être avec un exemple de code qui fait l'init de cozy-client avec cet attr à true, ça saute plus aux yeux je trouve.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ok je vais rajouter ça

This example was out of date with the new version. The next step is to add storybook to make a documentation page.
BREAKING CHANGE : You must have cozy-client >= 45.13.0
BREAKING CHANGE : You must have cozy-ui >= 103.7.0
The CSS preprocessor put an underscore before the double hyphen, making the classes obsolete. I followed the BEM convention and modified it to have a distinct class
@cballevre cballevre force-pushed the fix/v11-migration-returns branch from 1792480 to bf37f2e Compare February 29, 2024 15:32
@cballevre cballevre changed the title fix: Taking account of feedback following the migration of apps to 11 [VO-421] fix: Taking account of feedback following the migration of apps to 11 Feb 29, 2024
@cballevre cballevre merged commit 7e77733 into master Feb 29, 2024
2 checks passed
@cballevre cballevre deleted the fix/v11-migration-returns branch February 29, 2024 15:38
@cozy-bot
Copy link

🎉 This PR is included in version 12.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants