🐛 fix(url): Ensures url parts from branch names are valid (#4)

Author: TzviPM

CHANGELOG.md

@@ -7,6 +7,11 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [1.0.3] - 2023-10-27
+
+### Fixed
+- Url parts generated from branch names are now valid (#4).
+
 ## [1.0.2] - 2023-10-27
 
 ### Fixed
@@ -21,7 +26,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 This was the initial release of `laravel-deploy`.
 
-[unreleased]: https://github.com/TzviPM/laravel-deploy/compare/v1.0.2...HEAD
-[1.0.2]: https://github.com/TzviPM/laravel-deploy/compare/v1.0.0...v1.0.2
+[unreleased]: https://github.com/TzviPM/laravel-deploy/compare/v1.0.3...HEAD
+[1.0.3]: https://github.com/TzviPM/laravel-deploy/compare/v1.0.2...v1.0.3
+[1.0.2]: https://github.com/TzviPM/laravel-deploy/compare/v1.0.1...v1.0.2
 [1.0.1]: https://github.com/TzviPM/laravel-deploy/compare/v1.0.0...v1.0.1
 [1.0.0]: https://github.com/TzviPM/laravel-deploy/releases/tag/v1.0.0

dist/index.js

@@ -1,6 +1,6 @@
 {
   "name": "laravel-deploy",
-  "version": "1.0.2",
+  "version": "1.0.3",
   "description": "A GitHub Action to create on-demand preview environments for Laravel apps.",
   "author": "Tzvi Melamed (TzviPM)",
   "private": true,

package-lock.json

@@ -49,7 +49,7 @@ export class DeploymentService {
     const { id: serverId, domain: domain } = servers[0];
     const pr = this.contextService.getPullRequest();
     const branchName = pr.branchName;
-    const siteName = `${branchName}.${domain}`;
+    const siteName = `${urlName(branchName)}.${domain}`;
     const baseProjectName =
       this.configService.getProjectName() || titleCase(pr.repo.name);
     const projectName = `${baseProjectName} Preview - ${titleCase(branchName)}`;
@@ -186,3 +186,9 @@ function dbBranchName(branchName: string): string {
 
   return branchParts.map((part) => part.toLowerCase()).join('-');
 }
+
+function urlName(branchName: string): string {
+  const branchParts = branchName.split(/[-_]/);
+
+  return branchParts.map((part) => part.toLowerCase()).join('-');
+}