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

[pkg/sitemap-ext]: Fixes handling of projects with a base with a trailing slash #61

Merged
merged 3 commits into from
Mar 27, 2024

Conversation

Fryuni
Copy link
Owner

@Fryuni Fryuni commented Mar 27, 2024

No description provided.

Copy link

vercel bot commented Mar 27, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
inox-tools ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 27, 2024 11:04am

@Fryuni
Copy link
Owner Author

Fryuni commented Mar 27, 2024

/snapshot base-trailing-slash

Copy link
Contributor

Snapshots have been released for the following packages:

  • @inox-tools/aik-route-config@pr--base-trailing-slash
  • @inox-tools/sitemap-ext@pr--base-trailing-slash
  • @example/sitemap-ext@pr--base-trailing-slash
Publish Log
🦋  warn ===============================IMPORTANT!===============================
🦋  warn Packages will be released under the pr--base-trailing-slash tag
🦋  warn ----------------------------------------------------------------------
🦋  info npm info @inox-tools/aik-mod
🦋  info npm info @inox-tools/aik-route-config
🦋  info npm info @inox-tools/custom-routing
🦋  info npm info @inox-tools/inline-mod
🦋  info npm info @inox-tools/sitemap-ext
🦋  info npm info @inox-tools/velox-luna
🦋  warn @inox-tools/aik-mod is not being published because version 0.4.0 is already published on npm
🦋  info @inox-tools/aik-route-config is being published because our local version (0.0.0-base-trailing-slash-20240327001804) has not been published on npm
🦋  warn @inox-tools/custom-routing is not being published because version 0.2.1 is already published on npm
🦋  warn @inox-tools/inline-mod is not being published because version 1.3.1 is already published on npm
🦋  info @inox-tools/sitemap-ext is being published because our local version (0.0.0-base-trailing-slash-20240327001804) has not been published on npm
🦋  warn @inox-tools/velox-luna is not being published because version 0.2.0 is already published on npm
🦋  info Publishing "@inox-tools/aik-route-config" at "0.0.0-base-trailing-slash-20240327001804"
🦋  info Publishing "@inox-tools/sitemap-ext" at "0.0.0-base-trailing-slash-20240327001804"
🦋  success packages published successfully:
🦋  @inox-tools/[email protected]
🦋  @inox-tools/[email protected]
🦋  Creating git tags...
🦋  New tag:  @inox-tools/[email protected]
🦋  New tag:  @inox-tools/[email protected]
Build Log

> [email protected] build /home/runner/work/inox-tools/inox-tools
> turbo run --filter '@inox-tools/*' build

• Packages in scope: @inox-tools/aik-mod, @inox-tools/aik-route-config, @inox-tools/custom-routing, @inox-tools/inline-mod, @inox-tools/sitemap-ext, @inox-tools/velox-luna
• Running build in 6 packages
• Remote caching disabled
::group::@inox-tools/custom-routing:build
cache hit, suppressing logs 0ca0431b4407aa96
::endgroup::
::group::@inox-tools/inline-mod:build
cache hit, suppressing logs f5eec5c9f8a855cb
::endgroup::
::group::@inox-tools/aik-mod:build
cache hit, suppressing logs 1e502b13ed4b19cb
::endgroup::
::group::@inox-tools/velox-luna:build
cache hit, suppressing logs bcce0bd79bacbb06
::endgroup::
::group::@inox-tools/aik-route-config:build
cache miss, executing 0d3c26e64a090fc5

> @inox-tools/[email protected] build /home/runner/work/inox-tools/inox-tools/packages/aik-route-config
> tsup

CLI Building entry: src/index.ts
CLI Using tsconfig: tsconfig.json
CLI tsup v8.0.2
CLI Using tsup config: /home/runner/work/inox-tools/inox-tools/packages/aik-route-config/tsup.config.ts
CLI Target: node18
CLI Cleaning output folder
ESM Build start
ESM dist/index.js     9.80 KB
ESM dist/index.js.map 18.80 KB
ESM ⚡️ Build success in 106ms
DTS Build start
DTS ⚡️ Build success in 2675ms
DTS dist/index.d.ts 567.00 B
::endgroup::
::group::@inox-tools/sitemap-ext:build
cache miss, executing d84e8615b004e27a

> @inox-tools/[email protected] build /home/runner/work/inox-tools/inox-tools/packages/sitemap-ext
> tsup

CLI Building entry: index.ts
CLI Using tsconfig: tsconfig.json
CLI tsup v8.0.2
CLI Using tsup config: /home/runner/work/inox-tools/inox-tools/packages/sitemap-ext/tsup.config.ts
CLI Target: node18
CLI Cleaning output folder
ESM Build start
ESM dist/index.js     5.00 KB
ESM dist/index.js.map 9.24 KB
ESM ⚡️ Build success in 79ms
DTS Build start
DTS ⚡️ Build success in 2397ms
DTS dist/index.d.ts 566.00 B
::endgroup::

 Tasks:    6 successful, 6 total
Cached:    4 cached, 6 total
  Time:    7.512s 

@Fryuni
Copy link
Owner Author

Fryuni commented Mar 27, 2024

/snapshot base-trailing-slash

Copy link
Contributor

Snapshots have been released for the following packages:

  • @inox-tools/aik-route-config@pr--base-trailing-slash
  • @inox-tools/sitemap-ext@pr--base-trailing-slash
  • @example/sitemap-ext@pr--base-trailing-slash
Publish Log
🦋  warn ===============================IMPORTANT!===============================
🦋  warn Packages will be released under the pr--base-trailing-slash tag
🦋  warn ----------------------------------------------------------------------
🦋  info npm info @inox-tools/aik-mod
🦋  info npm info @inox-tools/aik-route-config
🦋  info npm info @inox-tools/custom-routing
🦋  info npm info @inox-tools/inline-mod
🦋  info npm info @inox-tools/sitemap-ext
🦋  info npm info @inox-tools/velox-luna
🦋  warn @inox-tools/aik-mod is not being published because version 0.4.0 is already published on npm
🦋  info @inox-tools/aik-route-config is being published because our local version (0.0.0-base-trailing-slash-20240327012437) has not been published on npm
🦋  warn @inox-tools/custom-routing is not being published because version 0.2.1 is already published on npm
🦋  warn @inox-tools/inline-mod is not being published because version 1.3.1 is already published on npm
🦋  info @inox-tools/sitemap-ext is being published because our local version (0.0.0-base-trailing-slash-20240327012437) has not been published on npm
🦋  warn @inox-tools/velox-luna is not being published because version 0.2.0 is already published on npm
🦋  info Publishing "@inox-tools/aik-route-config" at "0.0.0-base-trailing-slash-20240327012437"
🦋  info Publishing "@inox-tools/sitemap-ext" at "0.0.0-base-trailing-slash-20240327012437"
🦋  success packages published successfully:
🦋  @inox-tools/[email protected]
🦋  @inox-tools/[email protected]
🦋  Creating git tags...
🦋  New tag:  @inox-tools/[email protected]
🦋  New tag:  @inox-tools/[email protected]
Build Log

> [email protected] build /home/runner/work/inox-tools/inox-tools
> turbo run --filter '@inox-tools/*' build

• Packages in scope: @inox-tools/aik-mod, @inox-tools/aik-route-config, @inox-tools/custom-routing, @inox-tools/inline-mod, @inox-tools/sitemap-ext, @inox-tools/velox-luna
• Running build in 6 packages
• Remote caching enabled
::group::@inox-tools/custom-routing:build
cache hit, suppressing logs 0ca0431b4407aa96
::endgroup::
::group::@inox-tools/inline-mod:build
cache hit, suppressing logs f5eec5c9f8a855cb
::endgroup::
::group::@inox-tools/aik-mod:build
cache hit, suppressing logs 1e502b13ed4b19cb
::endgroup::
::group::@inox-tools/velox-luna:build
cache hit, suppressing logs bcce0bd79bacbb06
::endgroup::
::group::@inox-tools/aik-route-config:build
cache miss, executing 93d9b00a9a2c5985

> @inox-tools/[email protected] build /home/runner/work/inox-tools/inox-tools/packages/aik-route-config
> tsup

CLI Building entry: src/index.ts
CLI Using tsconfig: tsconfig.json
CLI tsup v8.0.2
CLI Using tsup config: /home/runner/work/inox-tools/inox-tools/packages/aik-route-config/tsup.config.ts
CLI Target: node18
CLI Cleaning output folder
ESM Build start
ESM dist/index.js     9.80 KB
ESM dist/index.js.map 18.80 KB
ESM ⚡️ Build success in 131ms
DTS Build start
DTS ⚡️ Build success in 2619ms
DTS dist/index.d.ts 567.00 B
::endgroup::
::group::@inox-tools/sitemap-ext:build
cache miss, executing 36d2eeef57561fef

> @inox-tools/[email protected] build /home/runner/work/inox-tools/inox-tools/packages/sitemap-ext
> tsup

CLI Building entry: index.ts
CLI Using tsconfig: tsconfig.json
CLI tsup v8.0.2
CLI Using tsup config: /home/runner/work/inox-tools/inox-tools/packages/sitemap-ext/tsup.config.ts
CLI Target: node18
CLI Cleaning output folder
ESM Build start
ESM dist/index.js     5.24 KB
ESM dist/index.js.map 9.68 KB
ESM ⚡️ Build success in 85ms
DTS Build start
DTS ⚡️ Build success in 2408ms
DTS dist/index.d.ts 566.00 B
::endgroup::

 Tasks:    6 successful, 6 total
Cached:    4 cached, 6 total
  Time:    8.044s 

Copy link
Collaborator

@Adammatthiesen Adammatthiesen left a comment

Choose a reason for hiding this comment

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

Worked great to resolve all the issues with SSR and Slugged pages.

Only other "non-related" bug to resolve, is the sitemap duplication bug from both sitemap integrations outputting

@Fryuni Fryuni force-pushed the fix/sitemap-base-root branch from fca3860 to 9a9fd73 Compare March 27, 2024 11:04
@Fryuni Fryuni enabled auto-merge (rebase) March 27, 2024 11:05
@Fryuni Fryuni merged commit a1c57c9 into main Mar 27, 2024
11 checks passed
@Fryuni Fryuni deleted the fix/sitemap-base-root branch March 27, 2024 11:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants