Skip to content

Conversation

@brionmario
Copy link
Member

@brionmario brionmario commented Jul 21, 2025

Purpose

This pull request introduces several updates to enhance issue and pull request templates, improve project initialization, update dependencies, and refactor the codebase. The changes include replacing outdated templates, updating configurations, and modernizing the code to align with the latest libraries and best practices.

Issue and Pull Request Template Updates:

  • Added new issue templates for reporting bugs (.github/ISSUE_TEMPLATE/bug.yml), requesting improvements (.github/ISSUE_TEMPLATE/improvement.yml), and proposing new features (.github/ISSUE_TEMPLATE/new_feature.yml) with structured fields and checklists. [1] [2] [3]
  • Replaced the outdated pull request template with a streamlined version that includes sections for purpose, related issues/PRs, checklist, and security checks (.github/pull_request_template.md).
  • Removed legacy templates (issue_template.md and pull_request_template.md) in favor of the new structured formats. [1] [2]

Project Initialization Enhancements:

  • Updated .tmplr.yml to improve prompts for initializing projects, including changes to naming conventions, version defaults, and fallback values. Removed unnecessary files during initialization.
  • Adjusted package.json to align with the updated initialization process, including dependency updates and template variable changes. [1] [2]

Dependency and Configuration Updates:

  • Upgraded dependencies in package.json to use the latest versions of React, Asgardeo libraries, and ESLint plugins.
  • Refactored eslint.config.js to use defineConfig and removed redundant plugin configurations for a cleaner setup. [1] [2]

Codebase Refactoring:

  • Updated src/App.jsx and src/main.jsx to replace @asgardeo/auth-react with @asgardeo/react and @asgardeo/react-router, simplifying authentication and routing logic. [1] [2]
  • Minor corrections and improvements in vite.config.js and README.md for better clarity and alignment with the updated libraries. (vite.config.jsL4-R4, Ff1ea3e0L1)

These changes collectively modernize the project, improve developer experience, and ensure alignment with current best practices.

Related Issues

Related PRs

  • N/A

Checklist

  • Manual test round performed and verified.
  • 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

- Changed package name in package.json from "{{ tmplr.package_name }}" to "{{ tmplr.name }}".
- Updated dependencies to use newer versions of React and Asgardeo packages.
- Removed pull request template file as it is no longer needed.
- Refactored App component to use react-router and ProtectedRoute from @asgardeo/react-router.
- Updated main entry point to use AsgardeoProvider for authentication.
- Fixed Vite configuration comment URL.
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.

1 participant