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

feat: change to del button #194

Merged
merged 2 commits into from
Oct 16, 2023
Merged

feat: change to del button #194

merged 2 commits into from
Oct 16, 2023

Conversation

sophiazlin
Copy link
Contributor

@sophiazlin sophiazlin commented Sep 30, 2023

remove from report button

Issue Number(s): #184

change added to report button to a remove from report button

Checklist

  • Button on the card in the digital library itself allows the card to be removed from the report
  • Button replaces the "Add to Report" button since we would only want a card to be removed once it has been added.

How to Test

Go to standards page and try to delete a standard from the report.
May be buggy.

@netlify
Copy link

netlify bot commented Sep 30, 2023

Deploy Preview for southface ready!

Name Link
🔨 Latest commit d2a73b1
🔍 Latest deploy log https://app.netlify.com/sites/southface/deploys/652cc82d85dce7000872a78f
😎 Deploy Preview https://deploy-preview-194--southface.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@sophiazlin sophiazlin linked an issue Sep 30, 2023 that may be closed by this pull request
2 tasks
@github-actions
Copy link

github-actions bot commented Sep 30, 2023

Deploy preview for southface ready!

✅ Preview
https://southface-1c9wr5z6j-bitsofgood.vercel.app

Built with commit d2a73b1.
This pull request is being automatically deployed with vercel-action

@avayedawadi avayedawadi merged commit 2dfaec4 into main Oct 16, 2023
6 checks passed
@avayedawadi avayedawadi deleted the sl/184-removeFromReport branch October 16, 2023 05:30
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.

[Feature] Remove from Report Button on Card
2 participants