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.

Sign up for GitHub

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

Jump to bottom

[Feature] Transfers added to admin & import from csv #723

Merged
merged 2 commits into from
Dec 30, 2023
Merged

[Feature] Transfers added to admin & import from csv #723

merged 2 commits into from
Dec 30, 2023

Conversation

franpb14
Copy link
Collaborator

Changes

  • Index page for transfers with this columns Id, Post, Comments, Source, Destination, Quantity, Created, Organization, Actions
  • CSV Working in the same way as users & posts
  • CSV without headers using these attributes source (member_uid, organization_id), source_type (member, organization), destination (same as source), destination_type, amount, reason (optional), post_id (optional)

@sseerrggii
Copy link
Contributor

sseerrggii commented Dec 25, 2023 •
edited
Loading

Tested on staging, works fine 🟢
Can we deploy it on production because we want to use it ASAP

@markets markets changed the title [Feature] Transfers addedto activeadmin & import from csv [Feature] Transfers added to admin & import from csv Dec 27, 2023
@markets
Copy link
Collaborator

markets commented Dec 27, 2023

Ok ✔️ I'll release it in the next days!

@markets markets merged commit de95e00 into develop Dec 30, 2023
2 of 4 checks passed
@markets markets deleted the feature/transfer-activeadmin branch December 30, 2023 19:09
@markets
Copy link
Collaborator

markets commented Dec 30, 2023 •
edited
Loading

@sseerrggii @franpb14 I pushed some refactoring and released as v4.5.0 🚀 💫

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@franpb14 @sseerrggii @markets