Skip to content

Add optional base URL path to environment#2904

Merged
dpwatrous merged 1 commit intomainfrom
dpwatrous/url-utils
May 14, 2024
Merged

Add optional base URL path to environment#2904
dpwatrous merged 1 commit intomainfrom
dpwatrous/url-utils

Conversation

@dpwatrous
Copy link
Member

@dpwatrous dpwatrous commented May 14, 2024

When using bonito in an environment with a base path (ie: pages that define a <base> tag), automatically normalize calls through HttpClient to include the base path if it is defined.

@codecov-commenter
Copy link

codecov-commenter commented May 14, 2024

Codecov Report

Attention: Patch coverage is 91.30435% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 67.26%. Comparing base (256154e) to head (458de4e).

❗ Current head 458de4e differs from pull request most recent head 6bbd50a. Consider uploading reports for the commit 6bbd50a to get more accurate results

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2904      +/-   ##
==========================================
+ Coverage   67.25%   67.26%   +0.01%     
==========================================
  Files        1259     1260       +1     
  Lines       34187    34210      +23     
  Branches     6292     6298       +6     
==========================================
+ Hits        22991    23012      +21     
- Misses      11032    11033       +1     
- Partials      164      165       +1     
Files Coverage Δ
...onito-core/src/environment/abstract-environment.ts 77.61% <100.00%> (+1.04%) ⬆️
...ackages/bonito-core/src/environment/environment.ts 100.00% <ø> (ø)
packages/bonito-core/src/http/fetch-http-client.ts 68.57% <100.00%> (+1.90%) ⬆️
packages/bonito-core/src/http/mock-http-client.ts 75.53% <100.00%> (+0.53%) ⬆️
packages/bonito-core/src/util/index.ts 100.00% <100.00%> (ø)
packages/bonito-core/src/util/url.ts 86.66% <86.66%> (ø)

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 256154e...6bbd50a. Read the comment docs.

@dpwatrous dpwatrous force-pushed the dpwatrous/url-utils branch from 50654b1 to 6de0516 Compare May 14, 2024 20:32
@dpwatrous dpwatrous force-pushed the dpwatrous/url-utils branch from 6de0516 to 6bbd50a Compare May 14, 2024 20:59
@dpwatrous dpwatrous merged commit 6981b90 into main May 14, 2024
@dpwatrous dpwatrous deleted the dpwatrous/url-utils branch May 14, 2024 22:08
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