Skip to content

feat(app): update design and functionality to match other conference apps and migrate google maps to leaflet#1821

Merged
brandyscarney merged 20 commits intomainfrom
bc/design-fixes
Apr 18, 2025
Merged

feat(app): update design and functionality to match other conference apps and migrate google maps to leaflet#1821
brandyscarney merged 20 commits intomainfrom
bc/design-fixes

Conversation

@brandyscarney
Copy link
Member

@brandyscarney brandyscarney commented Apr 16, 2025

I updated some of the page designs here to be consistent with the other conference apps and migrated from google maps to leaflet.

The best way to test this is probably to compare this branch against main and the other conference apps:

Main Preview: https://ionic-conference-app-git-main-ionic1.vercel.app/tutorial
Branch Preview: https://ionic-conference-app-git-bc-design-fixes-ionic1.vercel.app/

Vue PR: ionic-team/ionic-vue-conference-app#55
Vue Preview: https://ionic-vue-conference-app-git-bc-fix-navigation-an-ad46b9-ionic1.vercel.app/tutorial

React PR: ionic-team/ionic-react-conference-app#133
React Preview: https://ionic-react-conference-app-git-bc-fix-navigation-ionic1.vercel.app/tutorial

@vercel
Copy link

vercel bot commented Apr 16, 2025

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

Name Status Preview Comments Updated (UTC)
ionic-conference-app ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 17, 2025 8:35pm

@brandyscarney brandyscarney changed the title feat(app): update design to match other conference apps feat(app): update design to match other conference apps and migrate from google maps to leaflet Apr 16, 2025
@brandyscarney brandyscarney changed the title feat(app): update design to match other conference apps and migrate from google maps to leaflet feat(app): update design and functionality to match other conference apps and migrate google maps to leaflet Apr 16, 2025
@brandyscarney brandyscarney requested review from a team and christian-bromann and removed request for a team April 16, 2025 22:37
Copy link
Contributor

@thetaPC thetaPC left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

Copy link
Member

@ShaneK ShaneK left a comment

Choose a reason for hiding this comment

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

LGTM! 🚀

@brandyscarney brandyscarney merged commit cb6a0e5 into main Apr 18, 2025
4 checks passed
@brandyscarney brandyscarney deleted the bc/design-fixes branch April 18, 2025 14:32
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.

3 participants