Skip to content

Conversation

@tudoramariei
Copy link
Member

@tudoramariei tudoramariei commented Dec 18, 2025

Closes #597

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds clipboard functionality to copy cause URLs, allowing users to easily share cause links. The implementation includes a reusable copy-to-clipboard component and refactors the get_cause method signature across view classes for consistency.

Key changes:

  • New icon_and_text.html template component for copying URLs with an icon and text display
  • Integration of the clipboard component in the form title header alongside existing action buttons
  • Standardization of the get_cause method signature to use keyword-only arguments across all view classes

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File Description
backend/templates/v2/ngo-account/components/form-title.html Adds copy-to-clipboard button next to "View public page" button, includes minor formatting changes to SVG path elements
backend/templates/v2/components/copy-to-clipboard/icon_and_text.html New template component that extends the base clipboard component, displays a link icon with customizable text
backend/donations/views/ngo_account/my_organization.py Updates NgoMainCauseView.get_cause to match new signature with keyword-only parameters, removes redundant cause assignment (now handled by parent class)
backend/donations/views/ngo_account/causes.py Adds get_absolute_cause_url helper method and updates get_cause signature in NgoCauseCommonView and its child classes (NgoCauseCreateView, NgoCauseEditView)
Comments suppressed due to low confidence (1)

backend/donations/views/ngo_account/causes.py:191

  • Illegal class 'NoneType' raised; will result in a TypeError being raised instead.
        raise None

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@tudoramariei tudoramariei force-pushed the copy/form-link branch 2 times, most recently from 5c90b2b to d7d4dd0 Compare December 18, 2025 14:24
@tudoramariei tudoramariei merged commit f46598d into main Dec 18, 2025
9 checks passed
@tudoramariei tudoramariei deleted the copy/form-link branch December 18, 2025 14:58
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.

Add Copy form link button

2 participants