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

Update devenv-lists-as-tiles.md #3371

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

Update devenv-lists-as-tiles.md #3371

wants to merge 1 commit into from

Conversation

jswymer
Copy link
Contributor

@jswymer jswymer commented Sep 19, 2024

This PR is created to follow up on the question asked in #3367 regarding the note:

"If I understand this note correct then the sentence maybe should be split into both scenarios.

Field definied only on Table Object -> Field won't be shown on the tile Field is defined on the Page Object -> Field isn't visible to the user -> Field won't be shown in the tile view. To summarize: Field must be added to the Page Object and visible for the user. The last sentence in the current note mentions that the field should be added via personalization and then it's visible in the tile. If the field has an AccessByPermission defined on table level then this field won't be visible to users without that permission."

This PR is created to follow up on the question asked in #3367 regarding the note:

"If I understand this note correct then the sentence maybe should be split into both scenarios.

Field definied only on Table Object -> Field won't be shown on the tile
Field is definied on the Page Object -> Field isn't visible to the user -> Field won't be shown in the tile view.
To summarize: Field must be added to the Page Object and visible for the user.
The last sentence in the current note mentions that the field should be added via personalization and then it's visible in the tile.
If the field has an AccessByPerrmission defined on table level then this field won't be visible to users without that permission."
Copy link
Contributor

Learn Build status updates of commit c104ac1:

✅ Validation status: passed

File Status Preview URL Details
dev-itpro/developer/devenv-lists-as-tiles.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@jswymer
Copy link
Contributor Author

jswymer commented Sep 19, 2024

@NKarolak @pri-kise

I apologize for overlooking this question in the original issue before closing it. This PR will track this issue and allow for discussion.

@jswymer jswymer self-assigned this Sep 19, 2024
@jswymer jswymer added the investigating Used when the issue needs to be further investigated. label Sep 19, 2024
Copy link
Contributor

@NKarolak NKarolak left a comment

Choose a reason for hiding this comment

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

Thank you, John!
prikise has informed me that he's also fine with your suggestion, but he won't be on GitHub soon. Thus, this approval also represents him.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
investigating Used when the issue needs to be further investigated.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants