Skip to content
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

Hyperlink Analyses on ARs to their WSs if assigned #775

Merged
merged 3 commits into from
Apr 20, 2018
Merged

Hyperlink Analyses on ARs to their WSs if assigned #775

merged 3 commits into from
Apr 20, 2018

Conversation

juangallostra
Copy link
Contributor

Description of the issue/feature this PR addresses

Linked issue: #722

Current behavior before PR

The Worksheet Analyses are not indicated as such on the AR view. No icon linking to the worksheet appears.

Desired behavior after PR is merged

The Worksheet Analyses are indicated as such on the AR view with an icon linking to the worksheet.

--
I confirm I have tested this PR thoroughly and coded it according to PEP8
and Plone's Python styleguide standards.

@juangallostra juangallostra self-assigned this Apr 20, 2018
@juangallostra juangallostra changed the title Fix conditional check that looks for non analysis requests Hyperlink Analyses on ARs to their WSs if assigned Apr 20, 2018
@xispa xispa merged commit 2ecacca into senaite:master Apr 20, 2018
@juangallostra juangallostra deleted the worksheet-icon-analysis branch April 23, 2018 07:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants