Skip to content

Add ability to edit saved dependencies #286

Open
flickz wants to merge 2 commits intomainfrom
feat/edit-savedDependencies
Open

Add ability to edit saved dependencies #286
flickz wants to merge 2 commits intomainfrom
feat/edit-savedDependencies

Conversation

@flickz
Copy link
Collaborator

@flickz flickz commented Dec 9, 2019

Resolves opencollective/opencollective#2686

This PR gives loggedInUser ability to edit saved dependencies, it first check if the loggedInUser has saved dependencies and displays a banner as shown below in profile.js page, the banner is removable if the user doesn't care to edit.

Screenshot 2019-12-11 at 11 06 49 AM

Screenshot 2019-12-11 at 12 59 45 PM

@flickz flickz requested a review from znarf December 9, 2019 17:02
@vercel
Copy link

vercel bot commented Dec 9, 2019

This pull request is being automatically deployed with ZEIT Now (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://zeit.co/opencollective/backyourstack/vgnsmjafno
🌍 Preview: In Progress

feat(editSavedDependenecies): update edit saved selected dependencies
@codecov-io
Copy link

Codecov Report

Merging #286 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #286   +/-   ##
=======================================
  Coverage   50.19%   50.19%           
=======================================
  Files          15       15           
  Lines         259      259           
  Branches       49       49           
=======================================
  Hits          130      130           
  Misses        104      104           
  Partials       25       25

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 2a7b5f9...bf95c77. Read the comment docs.

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.

BackYourStack: update dependency selection

2 participants