Skip to content

Conversation

ihorsokhanexoft
Copy link
Contributor

Purpose

Components are ignored when user creates a view-only link and includes components

Ticket

https://openscience.atlassian.net/browse/ENG-8742?atlOrigin=eyJpIjoiYjg1ZGE2MzI3YjMyNGE2YWFiMDYyOGMyMjQyOTJhYmYiLCJwIjoiaiJ9

@ihorsokhanexoft ihorsokhanexoft changed the title don't ignore components when create a view-only link [ENG-8742] include ignored components when create a view-only link Aug 27, 2025
Copy link
Contributor

@Johnetordoff Johnetordoff left a comment

Choose a reason for hiding this comment

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

This should add a test case to ensure the logic stays intact, but just add a few additional assertions to an existing test and this PR is fine.

@Johnetordoff
Copy link
Contributor

@ihorsokhanexoft I was very surprised with your report of the tests passing locally, yet failing in the CI, but I was able to reproduce the phenomenon and confirmed you were correct. There is some issue with our code working with GHA, that seems like a separate issue. @adlius That might need it's our ticket, I can't see what in this PR could cause that local tests to pass. Maybe an issue with the base branch?

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.

2 participants