Skip to content

Conversation

@3nethz
Copy link
Contributor

@3nethz 3nethz commented Feb 26, 2025

Purpose

This PR initializes the Vue-Vite sample app with essential configurations:

  • Project Setup: Created the Vue 3 + Vite project structure.
  • ESLint & Prettier: Configured linting and formatting.
  • TypeScript: Set up tsconfig.json.
  • Package Management: Updated package.json with dependencies and scripts.

Related Issues

Related PRs

Checklist

  • e2e cypress tests locally verified.
  • Manual test round performed and verified.
  • UX/UI review done on the final implementation.
  • Documentation provided. (Add links if there are any)
  • Unit tests provided. (Add links if there are any)
  • Integration tests provided. (Add links if there are any)

Security checks

@@ -0,0 +1,45 @@
# vue-vite

This template should help get you started developing with Vue 3 in Vite.
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's update the readme to match the sample app

<meta charset="UTF-8" />
<link rel="icon" href="/favicon.ico" />
<meta name="viewport" content="width=device-width, initial-scale=1.0" />
<title>Vite App</title>
Copy link
Contributor

Choose a reason for hiding this comment

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

Updste the title

@@ -0,0 +1,42 @@
{
"name": "vue-vite",
Copy link
Contributor

Choose a reason for hiding this comment

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

Update app name

## Getting Started

### Prerequisites
- `Node.js` (version 10 or above).
Copy link
Contributor

Choose a reason for hiding this comment

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

Node 18 reaches EOL on April, 2025. Therefore no point in mentioning 10 (Vite and PNPM wouldn't even run on node v10). Let's update this to v20 or above.


## Contribute

Please read [Contributing to the Code Base](http://wso2.github.io/) for details on our code of conduct, and the process for submitting pull requests to us.
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's point to CONTRIBUTING.md in repo root.


### Reporting Issues

We encourage you to report issues, improvements, and feature requests by creating [Github Issues](https://github.com/asgardeo/asgardeo-auth-vue-sdk/issues).
Copy link
Contributor

Choose a reason for hiding this comment

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

name: 'app/files-to-lint',
files: ['**/*.{ts,mts,tsx,vue}'],
},

Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change

Copy link
Contributor

Choose a reason for hiding this comment

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

Let's use asgardeo icon

Copy link
Contributor

Choose a reason for hiding this comment

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

Let's use asgardeo logo

import vue from '@vitejs/plugin-vue'
import vueDevTools from 'vite-plugin-vue-devtools'

// https://vite.dev/config/
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
// https://vite.dev/config/

"jsdom": "^26.0.0",
"npm-run-all2": "^7.0.2",
"prettier": "^3.5.1",
"typescript": "~5.7.3",
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's update the TS version to workspace version

"compilerOptions": {
"noEmit": true,
"tsBuildInfoFile": "./node_modules/.tmp/tsconfig.node.tsbuildinfo",

Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change

"exclude": [],
"compilerOptions": {
"tsBuildInfoFile": "./node_modules/.tmp/tsconfig.vitest.tsbuildinfo",

Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change

@@ -0,0 +1,58 @@
<!--
* Copyright (c) 2025, WSO2 LLC. (https://www.wso2.com).
Copy link
Contributor

Choose a reason for hiding this comment

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

Remove the boilerplate files. Check others files as well

@3nethz 3nethz requested a review from DonOmalVindula March 6, 2025 08:57
pavinduLakshan
pavinduLakshan previously approved these changes Mar 7, 2025
@pavinduLakshan pavinduLakshan merged commit 293d05b into asgardeo:main Mar 7, 2025
0 of 4 checks passed
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