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

Add MART reports importer #5032

Draft
wants to merge 21 commits into
base: main
Choose a base branch
from
Draft

Add MART reports importer #5032

wants to merge 21 commits into from

Conversation

pacodelcastillolopez
Copy link
Collaborator

@pacodelcastillolopez pacodelcastillolopez commented Dec 18, 2024 •
edited by gjvoosten
Loading

Implementation of a MART reports importer and an admin GUI to monitor it.

Closes AB#1011

User changes

  • None

Superuser changes

  • None

Admin changes

  • Can see a log of the MART report importer in a new screen in the Admin area.

System admin changes

  • application.yml or anet-dictionary.yml needs change
  • db needs migration
  • documentation has changed
  • graphql schema has changed

Checklist

  • described the user behavior in PR body
  • referenced/updated all related issues
  • commits follow a repo#issue: Title title format and these 7 rules
  • commits have a clean history, otherwise PR may be squash-merged
  • added and/or updated unit tests
  • added and/or updated e2e tests
  • added and/or updated data migrations
  • updated documentation
  • resolved all build errors and warnings
  • opened debt issues for anything not resolved here

@pacodelcastillolopez pacodelcastillolopez added improvement MART Mobile Advisor Reporting Tool labels Dec 18, 2024
@gjvoosten gjvoosten changed the title MART reports importer Add MART reports importer Dec 18, 2024
@gjvoosten gjvoosten marked this pull request as draft December 19, 2024 10:22
@gjvoosten gjvoosten force-pushed the AB#1011-be-able-to-import-MART-reports branch 8 times, most recently from 318d11c to 8890569 Compare January 7, 2025 10:52
@gjvoosten gjvoosten force-pushed the AB#1011-be-able-to-import-MART-reports branch 2 times, most recently from 3b715be to 50c3553 Compare January 15, 2025 17:16
@gjvoosten gjvoosten force-pushed the AB#1011-be-able-to-import-MART-reports branch from 50c3553 to a8ba7d0 Compare January 16, 2025 15:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gjvoosten gjvoosten Awaiting requested review from gjvoosten

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
feature MART Mobile Advisor Reporting Tool
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@pacodelcastillolopez @gjvoosten