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

refactor(meta): use ngx-meta lib #218

Merged
merged 3 commits into from
Dec 17, 2023
Merged

Conversation

davidlj95
Copy link
Owner

@davidlj95 davidlj95 commented Nov 14, 2023

Uses freshly baked @davidlj95/ngx-meta package to set metadata. Instead of @ngaox/seo

The library has peer dependencies updated :) And tests to ensure it works across several Angular versions. Including new one (v17)

Copy link
Owner Author

Current dependencies on/for this PR:

This stack of pull requests is managed by Graphite.

angular.json Outdated Show resolved Hide resolved
@davidlj95 davidlj95 force-pushed the stacked/feat-seo-add-seo-services branch from 3bdbd77 to 1e2803a Compare December 17, 2023 17:09
@davidlj95 davidlj95 changed the title feat(metadata): add metadata services refactor(meta): use ngx-meta lib Dec 17, 2023
@davidlj95 davidlj95 force-pushed the stacked/feat-seo-add-seo-services branch from 1e2803a to 13fe428 Compare December 17, 2023 17:12
@davidlj95 davidlj95 marked this pull request as ready for review December 17, 2023 17:12
Comment on lines -1 to -6
<?xml version="1.0" encoding="UTF-8"?>
<project version="4">
<component name="JavaScriptLibraryMappings">
<file url="PROJECT" libraries="{decap-cms}" />
</component>
</project>
Copy link
Owner Author

Choose a reason for hiding this comment

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

No idea why this disappeared but I think it's not needed in first place

Copy link
Owner Author

Choose a reason for hiding this comment

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

It appeared when introducing Decap CMS in it's index.html but we barely touch that file so

@davidlj95 davidlj95 merged commit 8d9e604 into main Dec 17, 2023
6 checks passed
@davidlj95 davidlj95 deleted the stacked/feat-seo-add-seo-services branch December 17, 2023 17:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant