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

Feature/public config should suport dynamic values #2635

Merged
merged 38 commits into from
Nov 25, 2024

Conversation

Abhinegi2
Copy link
Contributor

@Abhinegi2 Abhinegi2 commented Oct 24, 2024

closes: #2401

Visible/Frontend Changes

  • Default value will be visible in public forms

@Abhinegi2 Abhinegi2 self-assigned this Oct 24, 2024
@Abhinegi2 Abhinegi2 marked this pull request as draft October 24, 2024 12:29
Copy link
Contributor

Deployed to https://pr-2635.aam-digital.net/

@sleidig sleidig requested a review from tomwwinter October 25, 2024 08:42
@Abhinegi2
Copy link
Contributor Author

let me resolve the conflicts.

Copy link
Contributor

@tomwwinter tomwwinter left a comment

Choose a reason for hiding this comment

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

Looks good so far. However, somehow the default values are not applied yet.

Next step would be to take a deeper look at EntityFormService.createEntityForm() and analyse, why the default value is not applied correctly.

@Abhinegi2 Abhinegi2 requested a review from tomwwinter November 6, 2024 10:47
@Abhinegi2
Copy link
Contributor Author

Looks good so far. However, somehow the default values are not applied yet.

Next step would be to take a deeper look at EntityFormService.createEntityForm() and analyse, why the default value is not applied correctly.

@tomwwinter I have updated the EntityFormService now the default value is working fine. You can test this further.

@Abhinegi2 Abhinegi2 marked this pull request as ready for review November 6, 2024 11:03
@Abhinegi2 Abhinegi2 requested a review from tomwwinter November 13, 2024 09:08
Copy link
Contributor

@tomwwinter tomwwinter left a comment

Choose a reason for hiding this comment

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

LGTM :)

@Abhinegi2 Abhinegi2 requested a review from sleidig November 18, 2024 12:03
sleidig added a commit that referenced this pull request Nov 20, 2024
while still including the angular form field for default values etc.

see #2635
@sleidig
Copy link
Member

sleidig commented Nov 20, 2024

This is ready to merge as soon as #2668 is available

sleidig added a commit that referenced this pull request Nov 21, 2024
while still including the angular form field for default values etc.

see #2635
sleidig added a commit that referenced this pull request Nov 21, 2024
while still including the angular form field for default values etc.

see #2635
@Abhinegi2 Abhinegi2 requested a review from sleidig November 21, 2024 14:34
@sleidig sleidig merged commit c02ed26 into master Nov 25, 2024
8 checks passed
@sleidig sleidig deleted the feature/public-config-should-suport-dynamic-values branch November 25, 2024 12:56
@aam-digital-ci
Copy link
Collaborator

🎉 This PR is included in version 3.43.0-master.8 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@aam-digital-ci aam-digital-ci added the released on @master managed by CI (semantic-release) label Nov 25, 2024
@aam-digital-ci
Copy link
Collaborator

🎉 This PR is included in version 3.43.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@aam-digital-ci aam-digital-ci added the released managed by CI (semantic-release) label Nov 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
released on @master managed by CI (semantic-release) released managed by CI (semantic-release)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Public Forms should support dynamic prefilled / default values
4 participants