-
Notifications
You must be signed in to change notification settings - Fork 104
Import Monitoring #3279
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
Draft
jacquesfize
wants to merge
32
commits into
develop
Choose a base branch
from
feat/import-monitorings
base: develop
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Import Monitoring #3279
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #3279 +/- ##
===========================================
+ Coverage 84.24% 84.25% +0.01%
===========================================
Files 131 131
Lines 10072 10146 +74
===========================================
+ Hits 8485 8549 +64
- Misses 1587 1597 +10
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
2b5f26f
to
dcc764e
Compare
719b85f
to
bfcf4a7
Compare
84786fc
to
c987a38
Compare
* feat(import): can post fieldmapping in new structured format * fix(import): insert_import_data_in_transient_table * fix(import): get_import_values * fix(import): prepare_import * feat(import): support default values * feat(import): default value edition support nomenclature widget refactor(import): minor refact on getFieldMappingValues() * feat(import): improve default value json edition * feat(import): present default values on report * feat(import): db migration * feat(import): test default value * test(import): support fieldmapping format v2 * feat(import): dynamic form on default values * fix(import): no longer send empty string for date default value * fix(import): test field jsonschema_definitions * feat(import): fix occhab revisions * feat(import): rename obsolete revision * style(import): code format * feat(import): fieldmapping validators deal with default values * feat(import): occhab revision to set bib_fields.type_field * Fix (migration) move migration to geonature branch --------- Co-authored-by: Pierre-Narcisi <pierre.narcisi@epitech.eu>
…ing model (#3275) * add id_import in monitoring model * feat: add id_import columns in monitoring tables * fix(db): change head of revision to latest in feat/import-monitorings --------- Co-authored-by: Julien Corny <julien_corny@natural-soltuions.eu> Co-authored-by: jacquesfize <jacques.fize@ecrins-parcnational.fr>
* feat: improve route resolver to import/:destinations with non existing destination * feat: add frontend test to route resolver ipdate
…d columns from t_imports_XXX
* feat: add modal for edit import action - create modal action component - call modal on edit import and next button step - add model modalData - remove deleteModal (replace by new component) [Refs]: https://github.com/orgs/PnX-SI/projects/13?pane=issue&itemId=82942301 Reviewed-by: andriacap * refact: add logic to check if import is done - Add method and accessor get to check if import is completed - Replace code where checking if import is completed Reviewed-by: andriacap * fix: adjust to dataset displacement * feat: add modal c onfirmation on frontend tests * lint: frontend tests * Apply suggestions from code review --------- Co-authored-by: Etienne Delclaux <etienne_delclaux@natural-solutions.eu> Co-authored-by: Jacques Fize <4259846+jacquesfize@users.noreply.github.com>
--------- Co-authored-by: Pierre-Narcisi <pierre.narcisi@epitech.eu> Co-authored-by: Etienne Delclaux <etienne_delclaux@natural-solutions.eu>
c987a38
to
54ac169
Compare
* link an entity to an object code * remove insecure on delete cascade * wip
* feat: modifications of fielammping: static, clean, dynalmic form wrapper * feat: default_value to constant * feat: test constant switch button * ui: edit constant button * feat: update fieldmapping step formulary * feat: improve field missing message * fix: add migration to update constant boolean from column_src * fix:adjust tests * fix: adapt test to new xor behaviuor on 'column_src' vs 'constant_value' * fix: frontend tests * fix: generate altitude in occhab * fix: attempt at fixing test frontend * test: force reload fieldmapping-input * fix: update actions coverage * update test cleaing * test ci * fix: date input in test * feat: remove nom_cite * minor fixes --------- Co-authored-by: jacquesfize <jacques.fize@ecrins-parcnational.fr>
--------- Co-authored-by: Vincent Bourgeois <vincent_bourgeois@natural-solutions.eu> Co-authored-by: jacquesfize <jacques.fize@ecrins-parcnational.fr>
* feat-import: make import compatible with single entity import fix: ci maybe fix: ci maybe * fix: remove common string algorithm fixed * fix: import also uuids (mandatory or generated) * fix: setparentline_no : remove extra parent * fix: child_entity --> entity
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR aggregates changes to GeoNature for the integration of the monitoring destination in the import module.