Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[PF-66] Model inheritance in JS #53

Open
wants to merge 4 commits into
base: release/3.0.0
Choose a base branch
from
Open

[PF-66] Model inheritance in JS #53

wants to merge 4 commits into from

Conversation

SamuelPalaj
Copy link

Description

  • implementation of model inheritance
  • refactor of import, export and domain objects to work with new petriflow xsd schema
  • added new inheritance tests
  • fixes to existing tests

Implements PF-66 Model inheritance in JS

Blocking Pull requests

There are no dependencies on other PR

How Has Been This Tested?

By tests

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • My changes have been checked with @...
  • I have commented my code, particularly in hard-to-understand areas
  • I have resolved all conflicts with the target branch of the PR
  • I have updated and synced my code with the target branch
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing tests pass locally with my changes:
    • Lint test
    • Unit tests
    • Integration tests
  • I have checked my contribution with code analysis tools:
  • I have made corresponding changes to the documentation:
    • Developer documentation
    • User Guides
    • Migration Guides

- added extension/parent attribute to petri net class
- added import and export of extension
- implementation of model inheritance
- refactor of import, export and domain objects to work with new petriflow xsd schema
- added new inheritance tests
- fixes to existing tests
@SamuelPalaj SamuelPalaj added the New feature A change that introduces new functionality label Feb 4, 2025
@SamuelPalaj SamuelPalaj self-assigned this Feb 4, 2025
# Conflicts:
#	src/lib/export/export-utils.ts
#	src/lib/import/import-utils.ts
#	src/lib/model/index.ts
- after merge fixes
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
New feature A change that introduces new functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant