Fix management command and Celery task #140

Merged
hansegucker merged 6 commits from 32-celery-task-not-properly-registered into master 2022-01-08 11:20:40 +01:00
hansegucker commented 2022-01-07 13:02:18 +01:00 (Migrated from edugit.org)

Closes #32, #33, #31

Closes #32, #33, #31
hansegucker commented 2022-01-07 13:02:19 +01:00 (Migrated from edugit.org)

assigned to @hansegucker

assigned to @hansegucker
hansegucker commented 2022-01-07 14:08:22 +01:00 (Migrated from edugit.org)

added 25 commits

  • 594656d6 - Catch not-configured database to make it possible to run some management commands without database
  • 8e66320d - Provide default templates as YAML and support args for field types
  • 948bb5a1 - Fix phone number parsing
  • d12743d9 - Don't limit import to specific models
  • 88d64767 - Use args for field types and fix match type handling
  • 9f17633a - Add regex and street/housenumber field types
  • 22d73a9a - Support suffixing email address local parts with domains
  • c3a1cd39 - Fix/add primary group field types
  • 84862b93 - Update template for Schild-NRW students
  • 0ed3fb04 - Update changelog
  • c5a60ca5 - Fix typo in changelog
  • 17310884 - Fix check for `email_domain` argument
  • fa4df581 - Update copyright
  • cfb15ad9 - Fixed argument parsing for management command
  • 6a2ab856 - Allow import of first column (configurable)
  • 9e83b617 - Update copyright
  • 2ddd99e7 - Allow duplicate spaces in between street and house numbers
  • 09dd1a67 - Align capitalisation of Schild-NRW with upstream
  • 6f3bc883 - Allow fully-qualified mail addresses even if email_domain is set
  • 967f70bd - Add template for Schild-NRW teachers
  • b4b8cb55 - Use newer version of ALekSIS-Core
  • 3dfd25f1 - Use ruamel.yaml as YAML library
  • 6b821040 - Define import task in tasks
  • 27fe8f28 - Fix management command for CSV import
  • 3b8c25fa - Merge branch '28-allow-providing-args-for-field-types' into 32-celery-task-not-properly-registered

Compare with previous version

added 25 commits <ul><li>594656d6 - Catch not-configured database to make it possible to run some management commands without database</li><li>8e66320d - Provide default templates as YAML and support args for field types</li><li>948bb5a1 - Fix phone number parsing</li><li>d12743d9 - Don&#39;t limit import to specific models</li><li>88d64767 - Use args for field types and fix match type handling</li><li>9f17633a - Add regex and street/housenumber field types</li><li>22d73a9a - Support suffixing email address local parts with domains</li><li>c3a1cd39 - Fix/add primary group field types</li><li>84862b93 - Update template for Schild-NRW students</li><li>0ed3fb04 - Update changelog</li><li>c5a60ca5 - Fix typo in changelog</li><li>17310884 - Fix check for `email_domain` argument</li><li>fa4df581 - Update copyright</li><li>cfb15ad9 - Fixed argument parsing for management command</li><li>6a2ab856 - Allow import of first column (configurable)</li><li>9e83b617 - Update copyright</li><li>2ddd99e7 - Allow duplicate spaces in between street and house numbers</li><li>09dd1a67 - Align capitalisation of Schild-NRW with upstream</li><li>6f3bc883 - Allow fully-qualified mail addresses even if email_domain is set</li><li>967f70bd - Add template for Schild-NRW teachers</li><li>b4b8cb55 - Use newer version of ALekSIS-Core</li><li>3dfd25f1 - Use ruamel.yaml as YAML library</li><li>6b821040 - Define import task in tasks</li><li>27fe8f28 - Fix management command for CSV import</li><li>3b8c25fa - Merge branch &#39;28-allow-providing-args-for-field-types&#39; into 32-celery-task-not-properly-registered</li></ul> [Compare with previous version](/AlekSIS/official/AlekSIS-App-CSVImport/-/merge_requests/65/diffs?diff_id=15697&start_sha=54d9b3935c84ac7be9b4cdb30824e8df81ff7383)
hansegucker commented 2022-01-07 14:36:12 +01:00 (Migrated from edugit.org)

added 1 commit

  • b226844c - Activate synchronous usage of management command

Compare with previous version

added 1 commit <ul><li>b226844c - Activate synchronous usage of management command</li></ul> [Compare with previous version](/AlekSIS/official/AlekSIS-App-CSVImport/-/merge_requests/65/diffs?diff_id=15702&start_sha=3b8c25faacd5d1148b5c30df56194ba332fcaa47)
hansegucker commented 2022-01-07 14:37:47 +01:00 (Migrated from edugit.org)

added 1 commit

  • d54aa9ee - Update changelog

Compare with previous version

added 1 commit <ul><li>d54aa9ee - Update changelog</li></ul> [Compare with previous version](/AlekSIS/official/AlekSIS-App-CSVImport/-/merge_requests/65/diffs?diff_id=15703&start_sha=b226844cc54b29ba6e37d67c7793c9734d5df5a2)
hansegucker commented 2022-01-07 14:39:42 +01:00 (Migrated from edugit.org)

changed the description

changed the description
hansegucker commented 2022-01-07 14:39:47 +01:00 (Migrated from edugit.org)

added 1 commit

Compare with previous version

added 1 commit <ul><li>03f1e05d - Update changelog</li></ul> [Compare with previous version](/AlekSIS/official/AlekSIS-App-CSVImport/-/merge_requests/65/diffs?diff_id=15705&start_sha=d54aa9ee291256e0bc5e4ed61dd9e51d3881b4d8)
hansegucker commented 2022-01-07 14:41:22 +01:00 (Migrated from edugit.org)

changed title from Draft: {-Resolve "Celery task not properly registered"-} to Draft: {+Fix management command and Celery task+}

changed title from **Draft: {-Resolve "Celery task not properly registered"-}** to **Draft: {+Fix management command and Celery task+}**
hansegucker commented 2022-01-07 14:54:45 +01:00 (Migrated from edugit.org)

~"workflow::blocked" !139

~"workflow::blocked" !139
hansegucker commented 2022-01-07 21:56:22 +01:00 (Migrated from edugit.org)

marked this merge request as ready

marked this merge request as **ready**
hansegucker commented 2022-01-07 21:56:27 +01:00 (Migrated from edugit.org)

assigned to @nik and unassigned @hansegucker

assigned to @nik and unassigned @hansegucker
hansegucker commented 2022-01-07 21:56:35 +01:00 (Migrated from edugit.org)

No longer blocked.

No longer blocked.
nik commented 2022-01-07 22:38:29 +01:00 (Migrated from edugit.org)

added 6 commits

  • 03f1e05d...2f819c7e - 5 commits from branch master
  • 3edf4815 - Merge branch 'master' into '32-celery-task-not-properly-registered'

Compare with previous version

added 6 commits <ul><li>03f1e05d...2f819c7e - 5 commits from branch <code>master</code></li><li>3edf4815 - Merge branch &#39;master&#39; into &#39;32-celery-task-not-properly-registered&#39;</li></ul> [Compare with previous version](/AlekSIS/official/AlekSIS-App-CSVImport/-/merge_requests/65/diffs?diff_id=15788&start_sha=03f1e05dd1f3bec323dbd37558df1a9ffa8f622b)
nik (Migrated from edugit.org) scheduled this pull request to auto merge when all checks succeed 2022-01-07 22:39:20 +01:00
nik commented 2022-01-07 23:16:15 +01:00 (Migrated from edugit.org)
from .tasks import import_csv                                                                                                                    
from .util.process import import_csv

def csv_import(request: HttpRequest) -> HttpResponse: 

I think this is the most creative series of re-definitions I have seen within… ever ;)!

```python from .tasks import import_csv from .util.process import import_csv def csv_import(request: HttpRequest) -> HttpResponse: ``` I think this is the most creative series of re-definitions I have seen within… ever ;)!
hansegucker commented 2022-01-08 11:14:10 +01:00 (Migrated from edugit.org)

aborted the automatic merge because source branch was updated

aborted the automatic merge because source branch was updated
hansegucker commented 2022-01-08 11:14:10 +01:00 (Migrated from edugit.org)

added 1 commit

  • 97d108c5 - Merge branch 'master' into '32-celery-task-not-properly-registered'

Compare with previous version

added 1 commit <ul><li>97d108c5 - Merge branch &#39;master&#39; into &#39;32-celery-task-not-properly-registered&#39;</li></ul> [Compare with previous version](/AlekSIS/official/AlekSIS-App-CSVImport/-/merge_requests/65/diffs?diff_id=15842&start_sha=3edf4815c66bb7f6b1208753da46479e99533bf4)
nik (Migrated from edugit.org) merged commit 560b63bedc into master 2022-01-08 11:20:40 +01:00
nik commented 2022-01-08 11:20:41 +01:00 (Migrated from edugit.org)

mentioned in commit 560b63bedc

mentioned in commit 560b63bedc478e5f201ec4eaa23a4a5488e30638
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
aleksis/AlekSIS-App-CSVImport!140
No description provided.