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

Resolves #4676 - Add import partners on getting started #4710

Conversation

AthiraKadampatta
Copy link
Contributor

Resolves #4676

Description

  • Adds Import Partners button beside the Add a Partner button on the Dashboard > Getting Started section > Partner Agencies step.
  • The Import Partners button behaves the same as as the "Import Partners" button on the Left hand Menu > Partner Agencies > All Partners, i.e it opens a modal where one can download the sample CSV or upload the CSV to import the partners.

Type of change

  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Modified the existing system specs to verify if the partner CTA on the Dashboard Getting started section has 2 links.

This has also been tested by following the below steps:

  • Logged in as org_admin2@example.com. This admin belongs to a bank that is brand-new.
  • Clicked "Add a Storage Location", provided a name and address, and clicked save.
  • Clicked "Dashboard" in the left hand menu to view the Partner Agencies step having the Import Partners button.

Screenshots

Screenshot 2024-10-08 at 11 46 30 AM Screenshot 2024-10-08 at 11 46 51 AM

…n beside Add a Partner link. Add system tests to ensure the Partner cta section has 2 links.
@AthiraKadampatta AthiraKadampatta force-pushed the 4676-add-import-partners-on-getting-started branch from 165923f to 1804eef Compare October 8, 2024 07:13
@AthiraKadampatta AthiraKadampatta changed the title 4676 add import partners on getting started Resolves #4676 - Add import partners on getting started Oct 8, 2024
Copy link
Collaborator

@cielf cielf left a comment

Choose a reason for hiding this comment

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

LGTM from a functional point of view. passing to @dorner for a technical check.

@cielf cielf requested a review from dorner October 10, 2024 18:22
@dorner
Copy link
Collaborator

dorner commented Oct 11, 2024

All good here!

@dorner dorner merged commit 9496887 into rubyforgood:main Oct 11, 2024
11 checks passed
Copy link
Contributor

@AthiraKadampatta: Your PR Resolves #4676 - Add import partners on getting started is part of today's Human Essentials production release: 2024.10.13.
Thank you very much for your contribution!

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 import partners option on getting started
3 participants