Skip to content
This repository has been archived by the owner on Aug 27, 2024. It is now read-only.

add service for layout context #116

Merged
merged 6 commits into from
Feb 27, 2023

Conversation

peterjokumsen
Copy link
Contributor

🎉 Description

Added service to allow for defining and updating layout context.

Fixes #46

🚀 Type of change

  • 🆕 New feature (non-breaking change which adds functionality)

🧪 How Has This Been Tested?

  • Demo-app updated to use new functionality and served locally
  • New specs created to confirm functionality works as intended

🧾 Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules

@peterjokumsen peterjokumsen added the enhancement 💡 New feature or request label Feb 26, 2023
@peterjokumsen peterjokumsen self-assigned this Feb 26, 2023
@github-actions
Copy link
Contributor

Azure Static Web Apps: Your stage site is ready! Visit it here: https://ashy-flower-0950bd703-116.westeurope.2.azurestaticapps.net

@peterjokumsen peterjokumsen force-pushed the feat/add-service-for-layout-context-46 branch from cb2fb1a to 8bb4868 Compare February 27, 2023 20:09
@sonarqubecloud
Copy link

[ng-libs | cv-layout] Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

90.9% 90.9% Coverage
0.0% 0.0% Duplication

@github-actions
Copy link
Contributor

Azure Static Web Apps: Your stage site is ready! Visit it here: https://ashy-flower-0950bd703-116.westeurope.2.azurestaticapps.net

@sonarqubecloud
Copy link

[ng-libs | Demo App] Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

87.5% 87.5% Coverage
0.0% 0.0% Duplication

@peterjokumsen peterjokumsen marked this pull request as ready for review February 27, 2023 20:16
@peterjokumsen peterjokumsen added this pull request to the merge queue Feb 27, 2023
Merged via the queue into main with commit afde27d Feb 27, 2023
@peterjokumsen peterjokumsen deleted the feat/add-service-for-layout-context-46 branch February 27, 2023 20:19
peterjokumsen added a commit that referenced this pull request Feb 27, 2023
## 🎉 Description

Added service to allow for defining and updating layout context.

Fixes #46

## 🚀 Type of change

- [x] 🆕 New feature (non-breaking change which adds functionality)

## 🧪 How Has This Been Tested?

- [x] Demo-app updated to use new functionality and served locally
- [x] New specs created to confirm functionality works as intended

## 🧾 Checklist:

- [x] My code follows the style guidelines of this project
- [x] I have performed a self-review of my own code
- [x] I have commented my code, particularly in hard-to-understand areas
- [x] I have made corresponding changes to the documentation
- [x] My changes generate no new warnings
- [x] I have added tests that prove my fix is effective or that my
feature works
- [x] New and existing unit tests pass locally with my changes
- [x] Any dependent changes have been merged and published in downstream
modules

BEGIN_COMMIT_OVERRIDE
chore(deps): remove decorate-angular postInstall

feat(layout): create layout-context service

To be used to provide context, to override context for layout when not provided.

feat(layout): use layout-context service for layout data

feat(layout): add static function to provide services

feat(demo-app): use layout-context service for context

chore(docs): update README to include latest update
END_COMMIT_OVERRIDE
peterjokumsen added a commit that referenced this pull request Feb 27, 2023
## 🎉 Description

Added service to allow for defining and updating layout context.

Fixes #46

## 🚀 Type of change

- [x] 🆕 New feature (non-breaking change which adds functionality)

## 🧪 How Has This Been Tested?

- [x] Demo-app updated to use new functionality and served locally
- [x] New specs created to confirm functionality works as intended

## 🧾 Checklist:

- [x] My code follows the style guidelines of this project
- [x] I have performed a self-review of my own code
- [x] I have commented my code, particularly in hard-to-understand areas
- [x] I have made corresponding changes to the documentation
- [x] My changes generate no new warnings
- [x] I have added tests that prove my fix is effective or that my
feature works
- [x] New and existing unit tests pass locally with my changes
- [x] Any dependent changes have been merged and published in downstream
modules

chore(deps): remove decorate-angular postInstall

feat(layout): create layout-context service
To be used to provide context, to override context for layout when not provided.

feat(layout): use layout-context service for layout data

feat(layout): add static function to provide services

feat(demo-app): use layout-context service for context

chore(docs): update README to include latest update
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement 💡 New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat(@chill-viking/layout): use a service to define layout context
2 participants