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.

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

feat: Extensibility of import and export notes processing #28

Open
azayati opened this issue Oct 24, 2022 · 4 comments
Open

feat: Extensibility of import and export notes processing #28

azayati opened this issue Oct 24, 2022 · 4 comments

Comments

@azayati
Copy link
Member

azayati commented Oct 24, 2022

Rationale

Currently, import and export notes processing is performed directly in notes codebase. That's why it is not possible to perform other pre import or post import operations from other modules.

1. Functional requirements

This feature must not change the current notes import and export operations.

2. Non Functional Requirements

  • Expected volume & Performance requirements

    • No change.
  • Security requirements (Authentication, Authorizations, Audit track / traceability, data retention…)

    • No modification with actual behavior
  • Extensibility requirements (Identified extension points, pluggable implementations...)

    • Implement NoteImportProcessor interface which with pre import and post import methods, this will make possible for each module to implement its own pre import and post import connector operations.
    • Implement NoteExportProcessor interface which with pre export and post export methods, this will make possible for each module to implement its own pre export and post export connector operations.
  • Configurability requirements (properties introduced, default settings, jobs, etc...)

    • No configuration
  • Requirements related to existing users data

    • No change
  • Requirements related to existing features (ex: Add a button to existing modules...)

    • No change.
  • Feature Flag (property name, default status, target audience, functional behavior)

    • No feature flag
  • Other NFR (ex: Use of VueX, Vue, Vuetify)

    • No change.

3. Impacts

  • Documentation to update?

    • No documentation to update
  • Training level / Change management needs

    • No change management needs

4. Software Architecture

Architectural impact has to be evaluate with a technical study

  • Security

    • No security change
  • Access (GUI, API…)

    • No change.
  • Services & processing

    • No change.
  • Data and persistence

    • No data model modification
  • Support for clustering

    • No change.
  • Support for multitenancy

    • No change.
  • Integrations

    • No change.
  • Migration strategy

    • No change.

5. Annexes

  • Reasoning that explain the technical choices
  • Impacts (ex: impact on public REST/Java API)
  • References (links...)
@srenault-meeds srenault-meeds moved this to Open in MIP Board Nov 29, 2022
@srenault-meeds srenault-meeds moved this from Open to Submitted in MIP Board Nov 29, 2022
@rdenarie
Copy link
Member

rdenarie commented Dec 2, 2022

@margondicco I put this mip in open status, as there is no priority on that

@rdenarie rdenarie moved this from Submitted to Open in MIP Board Dec 2, 2022
@srenault-meeds srenault-meeds moved this from Open to Draft in MIP Board May 11, 2023
@srenault-meeds srenault-meeds moved this from Draft to Open in MIP Board May 11, 2023
@plamarque
Copy link
Member

May be useful when we will support images in notes.

@srenault-meeds
Copy link
Member

I archive it for now as no news for it since 7 months

@azayati
Copy link
Member Author

azayati commented Feb 28, 2024

I archive it for now as no news for it since 7 months

@srenault-meeds I clicked by mistake on "restore", you can archive it again ?

@srenault-meeds srenault-meeds moved this from Open to Rejected in MIP Board Feb 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

No branches or pull requests

4 participants