Skip to content

Conversation

@blankdots
Copy link
Contributor

Description

URL strips path from base url so when deploying we are not getting the correct path

Related issues

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Changes Made

  • concatenate URLs instead of using base on so that path remains

Testing

  • Tests do not apply

Mentions

@blankdots blankdots self-assigned this Feb 15, 2023
Copy link
Contributor

@hannyle hannyle left a comment

Choose a reason for hiding this comment

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

Looks good to me! It's nice to know about this issue.

Could we have a Changelog update for this ?

@sampsapenna sampsapenna force-pushed the feature/download-and-decrypt-files branch 2 times, most recently from 62c9107 to 56694af Compare March 10, 2023 15:58
@sampsapenna
Copy link
Member

@blankdots #929 was rebased and had some new changes – meaning this branch needs to be rebased as well. I can do it in case you don't have time for it, but you probably have a better grasp on how to resolve any conflicts in this.

@blankdots
Copy link
Contributor Author

blankdots commented Mar 10, 2023

@blankdots #929 was rebased and had some new changes – meaning this branch needs to be rebased as well. I can do it in case you don't have time for it, but you probably have a better grasp on how to resolve any conflicts in this.

@sampsapenna should have merged this before rebasing, but I can try to see what has happened

@blankdots blankdots force-pushed the bugfix/redirect-in-deployment branch 2 times, most recently from b9d1767 to 89fd547 Compare March 10, 2023 19:57
@blankdots blankdots marked this pull request as draft March 10, 2023 20:02
URL strips path from base url so when deploying we are not getting the correct path
@blankdots blankdots force-pushed the bugfix/redirect-in-deployment branch from 89fd547 to 2571a7b Compare March 10, 2023 20:05
@blankdots blankdots marked this pull request as ready for review March 10, 2023 20:06
@sampsapenna sampsapenna merged commit f3cdaee into feature/download-and-decrypt-files Mar 13, 2023
@sampsapenna sampsapenna deleted the bugfix/redirect-in-deployment branch March 13, 2023 08: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.

4 participants