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(context): import export context in demo #1348

Merged

Conversation

aziz-access
Copy link
Contributor

What is the current behavior? (You can also link to an open issue here)

this behavior related to #1338

What is the new behavior?

Add context module in demo to test emport and export contexts

Does this PR introduce a breaking change? (check one with "x")

  • Yes
  • No

If this PR contains a breaking change, please describe the impact and migration path for existing applications:

Other information:

@aziz-access aziz-access added enhancement New features and improvements // Nouvelles fonctionnalités et améliorations in progress In progress // Développement en cours labels Aug 18, 2023
@aziz-access aziz-access self-assigned this Aug 18, 2023
@aziz-access aziz-access linked an issue Aug 18, 2023 that may be closed by this pull request
</igo-panel>
</div>
</mat-grid-tile>
</mat-grid-list>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Est-ce qu'on pourrait l'soler en dessous dans un autre composante d'exemple pour éviter de surcharger celui existant. Ex :

Suggested change
</mat-grid-list>
<app-example-viewer
title="Import/Export context"
codeFolder="context/context"
configFolder="contexts"
>
<igo-context-import-export [map]="map"> </igo-context-import-export>
</app-example-viewer>

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Après des tests l'exportation ne fonctionne pas.
image

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this bug resolved in PR #1322

demo/src/app/context/context/context.component.html Outdated Show resolved Hide resolved
@alecarn alecarn self-requested a review August 24, 2023 20:18
@alecarn
Copy link
Collaborator

alecarn commented Aug 24, 2023

@pelord Je merge tout de suite parce que c'est mineur comme changement. Si tu peux tout de même jeter un oeil, merci.

@alecarn alecarn requested a review from pelord August 24, 2023 20:20
@alecarn alecarn merged commit 41cfaa6 into next Aug 24, 2023
5 checks passed
@alecarn alecarn deleted the 1338-contexts-allow-import-and-export-contexts-in-lib-demo branch August 24, 2023 20:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New features and improvements // Nouvelles fonctionnalités et améliorations in progress In progress // Développement en cours
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Contexts: Allow import and export Contexts in lib demo
2 participants