Skip to content
This repository has been archived by the owner on Jul 27, 2022. It is now read-only.

ISAICP-6408: Show unpublished releases in the unpublished section for solutions. #2446

Merged
merged 13 commits into from
May 11, 2021

Conversation

idimopoulos
Copy link
Contributor

No description provided.

@idimopoulos idimopoulos changed the title ISAICP-6408: Provide a test for the unpublished releases. ISAICP-6408: Show unpublished releases in the unpublished section for solutions. Apr 12, 2021
Comment on lines 179 to 181
# The owner of the content did not create the release as they are simple members.
And I should not see the following tiles in the "Unpublished content area" region:
| Hidden spies |
Copy link
Contributor

Choose a reason for hiding this comment

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

This seems like the correct place to test the functionality that is being asked in the ticket: to ensure that the author of an unpublished release can see it. This doesn't seem to not prove what is requested.

Comment on lines 200 to 203
# Releases are available.
And I should see the following tiles in the "Unpublished content area" region:
| The Ragged Streams |
| Hidden spies |
Copy link
Contributor

Choose a reason for hiding this comment

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

This doesn't seem to be correct, the ticket does not mention that facilitators should be able to see unpublished drafts of releases. It goes against the established pattern "The facilitator should not be able to see content that only have a draft state".

It is requested that authors should see their own releases, not facilitators.

Copy link
Contributor

@pfrenssen pfrenssen left a comment

Choose a reason for hiding this comment

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

Great! Now everything is clear. I added some comments to make things easier to follow.

@claudiu-cristea claudiu-cristea merged commit 336dd1a into develop May 11, 2021
@claudiu-cristea claudiu-cristea deleted the ISAICP-6408 branch May 11, 2021 12:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants