-
Notifications
You must be signed in to change notification settings - Fork 0
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
Mapping settings added to import settings #74
Conversation
|
|
|
from typing import Dict, List | ||
from django.db.models import Q | ||
|
||
# from apps.mappings.helpers import schedule_or_delete_fyle_import_tasks |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is fixed in next PR
) | ||
mapping_settings = validated.pop('mapping_settings') | ||
import_settings = validated.pop('import_settings') | ||
# dependent_field_settings = validated.pop('dependent_field_settings') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is fixed in next PR
# dependent_field_settings = validated.pop('dependent_field_settings') | ||
|
||
with transaction.atomic(): | ||
print("inside function") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
removed in next PR
No description provided.