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

Feature/203 vite #298

Merged
merged 52 commits into from
Apr 6, 2023
Merged

Feature/203 vite #298

merged 52 commits into from
Apr 6, 2023

Conversation

patric-eberle
Copy link
Member

Pull request

This PR replaces the webpack builder with Vite. Additionally it improves the TypeScript integration to get better feedback about invalid types. It also extends the test scripts with the TypeScript test. Several packages could be removed because of this change. I also switched the Storybook builder to Vite for better compatibility.

Ticket

#203

Browser testing

  • link

Checklist

  • I merged the current development branch (before testing)
  • Added JSDoc and styleguide demo
  • Tested all links in project relevant browsers
  • Tested all links in different screen sizes
  • Did run automated tests and linters
  • Did assign ticket
  • Double checked target branch

Review/Test checklist

  • Did review code and documentation
  • Tested all links in project relevant browsers
  • Tested all links in different screen sizes
  • Did check accessibility (Wave, only errors)
  • Re-assign ticket to developer

# Conflicts:
#	.eslintrc.js
#	package.json
#	src/setup/i18n.ts
#	src/stores/plugins/api.ts
#	src/types/index.d.ts
#	src/types/initial-data.d.ts
#	src/types/initial-data.ts
…aged config to separate file for better TSC support
# Conflicts:
#	package-lock.json
#	src/assets/icons.svg
#	src/types/image.d.ts
# Conflicts:
#	package-lock.json
#	package.json
#	src/stores/session.ts
#	src/styleguide/components/s-demo-settings.vue
#	webpack.config.js
#   .eslint.js
@patric-eberle patric-eberle added enhancement New feature or request vue-3 Will only be done for the vue-3 version WIP labels Feb 9, 2023
# Conflicts:
#	src/components/e-table.vue
#	src/styleguide/routes/r-google-maps.vue
#	webpack.config.js
#   src/stores/session.ts
# Conflicts:
#	src/components/e-progress.vue
#	src/directives/outside-click.ts
#	src/stores/session.ts
#	webpack.config.js
@patric-eberle
Copy link
Member Author

@MB-web can you please check my changes? I was also able to remove additional packages 🎉

@MB-web
Copy link
Collaborator

MB-web commented Feb 15, 2023

@patric-eberle Changes are fine, can be merged from my side.

I have updated lastDependencyUpdate in package.json, since you did an Update.

@patric-eberle
Copy link
Member Author

@patric-eberle Changes are fine, can be merged from my side.

I have updated lastDependencyUpdate in package.json, since you did an Update.

@MB-web thank you for the update. Unfortunately we will also have to write a new plugin for vite... else no message will be shown anymore.

@patric-eberle patric-eberle removed the WIP label Apr 6, 2023
@patric-eberle patric-eberle changed the title WIP: Feature/203 vite Feature/203 vite Apr 6, 2023
@patric-eberle patric-eberle merged commit 636af3f into develop Apr 6, 2023
@patric-eberle patric-eberle deleted the feature/203_vite branch April 6, 2023 11:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request vue-3 Will only be done for the vue-3 version
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants