Skip to content
This repository was archived by the owner on Feb 26, 2025. It is now read-only.

Disable data cart download button when no perms - #997

Merged
dhaneshnm merged 4 commits into
1.7.0-M3from
2926-disable-data-cart-download-button-when-no-permissions
Dec 13, 2021
Merged

dhaneshnm merged 4 commits into
1.7.0-M3from
2926-disable-data-cart-download-button-when-no-permissions

Conversation

@nicwells

@nicwells nicwells commented Nov 24, 2021

Copy link
Copy Markdown
Contributor

Fixes BlueBrain/nexus#2926

Description

Disables download button if user doesn't have the required permissions to download all the resources in their cart.

Note: requires the updated AccessControl component in the react-nexus package, BlueBrain/nexus-js#369

How has this been tested?

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added necessary unit and integration tests.
  • I have added screenshots (if applicable), in the comment section.

@nicwells
nicwells force-pushed the 2926-disable-data-cart-download-button-when-no-permissions branch from c12b476 to 31196e3 Compare November 24, 2021 08:34
@nicwells
nicwells marked this pull request as draft November 24, 2021 08:34
@dhaneshnm
dhaneshnm changed the base branch from 1.7.0-M2 to 1.7.0-M3 November 24, 2021 10:00
@nicwells
nicwells force-pushed the 2926-disable-data-cart-download-button-when-no-permissions branch from 31196e3 to b66c6df Compare December 8, 2021 14:50
@nicwells
nicwells marked this pull request as ready for review December 8, 2021 14:51
@nicwells
nicwells requested a review from dhaneshnm December 8, 2021 14:51
@codecov-commenter

codecov-commenter commented Dec 8, 2021

Copy link
Copy Markdown

Codecov Report

❗ No coverage uploaded for pull request base (1.7.0-M3@b0adcb7). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##             1.7.0-M3     #997   +/-   ##
===========================================
  Coverage            ?   66.00%           
===========================================
  Files               ?       21           
  Lines               ?      609           
  Branches            ?      126           
===========================================
  Hits                ?      402           
  Misses              ?      207           
  Partials            ?        0           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b0adcb7...1c31a42. Read the comment docs.

@dhaneshnm
dhaneshnm merged commit dd4eedf into 1.7.0-M3 Dec 13, 2021
@dhaneshnm
dhaneshnm deleted the 2926-disable-data-cart-download-button-when-no-permissions branch December 13, 2021 08:11
dhaneshnm pushed a commit that referenced this pull request Mar 11, 2022
* Disable data cart download button when no perms

* Specify correct version of nexus sdk and react nexus packages

Co-authored-by: Dhanesh Neela Mana <dhanesh.neelamana@epfl.ch>
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