Skip to content

docs(slides): update Angular migration guide to be compatible with Swiper 9 #2828

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

Merged
merged 3 commits into from
Mar 15, 2023

Conversation

averyjohnston
Copy link
Contributor

@averyjohnston averyjohnston commented Mar 14, 2023

Shortcut: https://ionic-docs-git-fw-3513-angular-ionic1.vercel.app/docs/angular/slides

This was a more significant overhaul than the React and Vue migrations because Swiper has removed its Angular component completely, in favor of Swiper Element.

Updating the example repo linked at the bottom of the docs will be done later. Those changes will be merged into main at the same time as this one.

If you want to play around with Swiper Element, here's the repo I've been using to test. Note that you'll also need this fix in Swiper for any of the functionality to work within an Ionic app; at the time of writing, this fix has been merged into main but not officially released. (I've written this guide with the assumption that it'll be released by the time our new docs go live.)

Here are the steps I wound up having to take to get the fix into the test repo:

  1. Clone https://github.com/nolimits4web/swiper.git
  2. In the cloned repo, run npm install && npm run build.
  3. Move into the “dist” directory and run npm pack. This will create a compressed archive of the built project called “swiper-9.1.0.tgz”.
  4. In the Ionic app, run npm install [path to compressed archive]
  5. restart the ionic app/dev server

@vercel
Copy link

vercel bot commented Mar 14, 2023

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

Name Status Preview Comments Updated
ionic-docs ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Mar 15, 2023 at 2:52PM (UTC)

@averyjohnston averyjohnston marked this pull request as ready for review March 14, 2023 20:09
@averyjohnston averyjohnston requested a review from a team March 14, 2023 20:09
Copy link
Contributor

@liamdebeasi liamdebeasi left a comment

Choose a reason for hiding this comment

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

Great job!

@averyjohnston averyjohnston merged commit f1984df into FW-3513 Mar 15, 2023
@averyjohnston averyjohnston deleted the FW-3513-angular branch March 15, 2023 15:16
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.

2 participants