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

Feature: Campaign Goal Block #7702

Open
wants to merge 23 commits into
base: epic/campaigns
Choose a base branch
from

Conversation

alaca
Copy link
Member

@alaca alaca commented Jan 31, 2025

Description

This PR is based on #7687 which should be reviewed first.

This pull request introduces several new features and improvements to the campaign functionality, including the addition of a new block for the campaign goal, as well as enhancements to existing components. The most significant changes include the implementation of the new block and updates to the campaign model and controller.

Testing Instructions

Pre-review Checklist

  • Acceptance criteria satisfied and marked in related issue
  • Relevant @unreleased tags included in DocBlocks
  • Includes unit tests
  • Reviewed by the designer (if follows a design)
  • Self Review of code and UX completed

@alaca alaca requested a review from pauloiankoski February 3, 2025 12:03
alaca added 3 commits February 3, 2025 13:09
…lock-GIVE-1510

# Conflicts:
#	src/Campaigns/Blocks/blocks.ts
#	src/Campaigns/Controllers/CampaignRequestController.php
src/Campaigns/Blocks/CampaignGoal/edit.tsx Outdated Show resolved Hide resolved
src/Campaigns/Blocks/shared/hooks/useCampaign.ts Outdated Show resolved Hide resolved
src/Campaigns/Blocks/CampaignGoal/styles.scss Outdated Show resolved Hide resolved
src/Campaigns/Blocks/CampaignGoal/edit.tsx Outdated Show resolved Hide resolved
@alaca
Copy link
Member Author

alaca commented Feb 3, 2025

@pauloiankoski this is ready for review

@alaca alaca requested a review from pauloiankoski February 3, 2025 16:36
@alaca
Copy link
Member Author

alaca commented Feb 7, 2025

@pauloiankoski this is ready for review.

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.

2 participants