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

feat(SearchBar): fetch and open note once clicked on a suggestion #773

Merged
merged 2 commits into from
Aug 22, 2022

Conversation

trollepierre
Copy link
Contributor

@trollepierre trollepierre commented Aug 18, 2022

Problem was detected by:
cozy/cozy-drive#2663

notes URL were not used to redirect, once clicked.

By fixing that bug, I realized the postMessage onSelect function could not be sent through postMessage. Becasue postMessage serializes the object before sending it.
An object could not be cloned

Fetching URL is now done inside Cozy-Bar scope

@trollepierre trollepierre changed the base branch from master to v7-stable August 18, 2022 08:25
@trollepierre trollepierre marked this pull request as ready for review August 18, 2022 08:52
@trollepierre trollepierre marked this pull request as draft August 18, 2022 09:29
@trollepierre trollepierre marked this pull request as ready for review August 18, 2022 12:18
@trollepierre trollepierre changed the title Search bar redirect not v7 feat(SearchBar): fetch and open note once clicked on a suggestion Aug 18, 2022
@trollepierre trollepierre force-pushed the search-bar-redirect-not-v7 branch 2 times, most recently from ae42465 to ed32b71 Compare August 22, 2022 09:20
@trollepierre trollepierre enabled auto-merge (rebase) August 22, 2022 12:13
@trollepierre
Copy link
Contributor Author

@Crash-- : you have approuved #774 but this is the PR that adds the feature in v7-stable branch. Could you approuved it?

@trollepierre trollepierre merged commit cee6acb into v7-stable Aug 22, 2022
@trollepierre trollepierre deleted the search-bar-redirect-not-v7 branch August 22, 2022 13:29
@cozy-bot
Copy link

🎉 This PR is included in version 7.20.1 🎉

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