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(components): expose the placeholder service in @o3r/components #1621

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

kpanot
Copy link
Contributor

@kpanot kpanot commented Apr 10, 2024

Proposed change

Expose the service to provide Placeholder Template URLs in the @o3r/components package instead on the rules engine.
The next (followed by #1617) is let the Effect also accessible via @o3r/components

Related issues

@kpanot kpanot requested a review from a team as a code owner April 10, 2024 14:25
@kpanot kpanot force-pushed the feature/placeholder-expose-service branch from 232ea58 to 151f16a Compare April 10, 2024 14:26
Copy link

nx-cloud bot commented Apr 10, 2024

☁️ Nx Cloud Report

CI is running/has finished running commands for commit a3105cc. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this CI Pipeline Execution


✅ Successfully ran 1 target

Sent with 💌 from NxCloud.

@kpanot kpanot marked this pull request as draft April 10, 2024 14:52
@kpanot kpanot force-pushed the feature/placeholder-expose-service branch 3 times, most recently from 30d0655 to b47b02c Compare April 15, 2024 17:06
@kpanot kpanot marked this pull request as ready for review April 15, 2024 17:13
@kpanot kpanot force-pushed the feature/placeholder-expose-service branch 3 times, most recently from e1c30a4 to ce47122 Compare April 19, 2024 08:06
@kpanot kpanot force-pushed the feature/placeholder-expose-service branch from ce47122 to f44d554 Compare April 19, 2024 08:49
@kpanot kpanot force-pushed the feature/placeholder-expose-service branch 2 times, most recently from 42d8a98 to 32a5ea7 Compare April 22, 2024 01:50
@kpanot kpanot force-pushed the feature/placeholder-expose-service branch from 32a5ea7 to a3105cc Compare April 22, 2024 03:15
@kpanot kpanot marked this pull request as draft April 22, 2024 03:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature]: Add the capability to use the placeholder without the Rules Engine
3 participants