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

Load js asynchronously and login using url #60

Merged
merged 7 commits into from
Jun 21, 2024
Merged

Conversation

MAX-786
Copy link
Member

@MAX-786 MAX-786 commented Jun 20, 2024

Fixes #47

This also fixes slow loading of iframe by cleaning up residue events which are being accumulated.

@MAX-786 MAX-786 requested a review from djay June 20, 2024 05:34
README.md Show resolved Hide resolved
README.md Outdated
@@ -73,50 +73,32 @@ To test against a local hydra instance
### Make your frontend editable

- Take the latest [hydra.js](https://github.com/collective/volto-hydra/tree/hydra.js) and include it in your frontend
Copy link
Member

Choose a reason for hiding this comment

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

this is repeated below. don't need both

Copy link
Member Author

Choose a reason for hiding this comment

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

we didn't mentioned about including hydra.js in below sections?

Copy link
Member

Choose a reason for hiding this comment

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

@MAX-786 This will need updating to use the @volto-hydra/hydra-js package now

Copy link
Member Author

Choose a reason for hiding this comment

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

this means directing them to packages/hydra-js , right? as this package is not made available they can't import it in their frontend.

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@MAX-786 MAX-786 requested a review from djay June 20, 2024 08:35
# Conflicts:
#	packages/hydra-js/hydra.js
#	packages/volto-hydra/src/components/Iframe/View.jsx
@JeffersonBledsoe
Copy link
Member

@MAX-786 I've merged main into this branch. Could you make sure this all still works as you intended before I review it?

@MAX-786
Copy link
Member Author

MAX-786 commented Jun 20, 2024

@JeffersonBledsoe I was fixing conflicts on local, THANKS for doing it . I'll just give it a final check.

@MAX-786
Copy link
Member Author

MAX-786 commented Jun 20, 2024

@JeffersonBledsoe LGTM! all features expect select block works fine

@MAX-786
Copy link
Member Author

MAX-786 commented Jun 20, 2024

Can we First merge this on main, so then I'll pull these changes to Select Block PR. Which will resolve your commented out changes there

@JeffersonBledsoe
Copy link
Member

@djay Can you check the docs again on this and I'll peak at the code?

@djay djay merged commit 4bb576a into main Jun 21, 2024
1 check passed
@djay djay deleted the load-js-and-login-using-url branch June 21, 2024 03:54
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.

As an integrator I can not have to load hydra.js all the time to know user has started editing
3 participants