-
Notifications
You must be signed in to change notification settings - Fork 7
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
Hotfix for failing linking to project search #147
Conversation
Passing run #168 ↗︎
Details:
Review all test suite changes for PR #147 ↗︎ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Testing the changes locally, and this does not work. It still shows 11 results when navigating the the link /about/projects?consortiaType=SPARC&search=implantable&selectedProjectsAnatomicalFocusIds=Vagus
I thought it was working, sorry... I made another change to avoid executing the Screen.Recording.2024-07-02.at.13.58.13.movNote that I cannot load the home page... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This breaks the consortia styling
components/FilesTable/FilesTable.vue
Outdated
@@ -680,7 +680,10 @@ export default { | |||
const fileType = scope.row.fileType.toLowerCase() | |||
const contentType = contentTypes[fileType] | |||
|
|||
const requestUrl = `${this.$config.public.portal_api}/download?s3BucketName=${s3BucketName}&key=${filePath}&contentType=${contentType}` | |||
const requestUrl = new URL(this.$config.public.portal_api + '/download') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think you can revert these changes, they do not appear related
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please revert these changes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works!
Tackles https://www.wrike.com/open.htm?id=1408704796