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.

Sign up for GitHub

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

Jump to bottom

Migrate to ionic 7 #243

Closed
5 of 6 tasks
ymaheshwari1 opened this issue Dec 26, 2023 · 0 comments · Fixed by #244
Closed
5 of 6 tasks

Migrate to ionic 7 #243

ymaheshwari1 opened this issue Dec 26, 2023 · 0 comments · Fixed by #244
Assignees
@ymaheshwari1

Comments

@ymaheshwari1
Copy link
Contributor

ymaheshwari1 commented Dec 26, 2023 •
edited
Loading

What is the motivation for adding/enhancing this feature?

Migrating to ionic 7.x will help in using some latest features in the components and will also solve some issues that were present in ionic 6.

What are the acceptance criteria?

  • Update to ionic 7
  • Migrate all the dependent package to a version that supports ionic 7
  • Update the components as per the migration guide
  • Test the components so that there is no breaking change

Can you complete this feature request by yourself?

  • YES
  • NO

Additional information
@ymaheshwari1 ymaheshwari1 self-assigned this Dec 26, 2023
ymaheshwari1 added a commit to ymaheshwari1/inventorycount that referenced this issue Dec 26, 2023
@ymaheshwari1
Improved: app to upgrade ionic to v7.6.0(hotwax#243)
63e76b4
ymaheshwari1 added a commit to ymaheshwari1/inventorycount that referenced this issue Dec 26, 2023
@ymaheshwari1
Improved: ion-select and ion-toggle components to remove handling of …
3e05d9e 
…programatic change and removed usage for ion-label for both these components(hotwax#243)
ymaheshwari1 added a commit to ymaheshwari1/inventorycount that referenced this issue Dec 26, 2023
@ymaheshwari1
Removed: unwanted check for qoh value before updating(hotwax#243)
ca5b0ac
@ymaheshwari1 ymaheshwari1 mentioned this issue Dec 26, 2023
Merged
1 task
ymaheshwari1 added a commit to ymaheshwari1/inventorycount that referenced this issue Dec 26, 2023
@ymaheshwari1
Improved: usage of ion-input to use label property and remove ion-lab…
5d9a029 
…el component for the same(hotwax#243)
ymaheshwari1 added a commit to ymaheshwari1/inventorycount that referenced this issue Dec 27, 2023
@ymaheshwari1
Improved: ion-radio code to use the label as per the changes in the l…
cd415f9 
…atest version(hotwax#243)
ymaheshwari1 added a commit to ymaheshwari1/inventorycount that referenced this issue Dec 27, 2023
@ymaheshwari1
Improved: apps theme package version to latest release(hotwax#243)
f4911b1
ymaheshwari1 added a commit to ymaheshwari1/inventorycount that referenced this issue Dec 27, 2023
@ymaheshwari1
Improved: package lock file as per the changes in the package(hotwax#243
777c870 
)
This was referenced Dec 27, 2023
Open
Closed
Closed
ravilodhi added a commit that referenced this issue Dec 28, 2023
@ravilodhi
Merge pull request #244 from ymaheshwari1/#243
c6c44ff 
Improved: app to upgrade ionic to v7.6.0(#243)
This was referenced Dec 28, 2023
Closed
Closed
This was referenced Apr 29, 2024
Closed
Merged
ymaheshwari1 added a commit that referenced this issue Apr 29, 2024
@ymaheshwari1
Merge pull request #278 from R-Sourabh/#243-html-template
8490df5 
Fixed: html tags not working in the translations (#243)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ymaheshwari1 ymaheshwari1

Labels
None yet
Projects
Digital Experience Platform
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Improved: app to upgrade ionic to v7.6.0(#243) ymaheshwari1/inventorycount
1 participant
@ymaheshwari1