Skip to content

Added resource locator strategy to portal configuration #15

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

Merged
merged 1 commit into from
Nov 12, 2013

Conversation

danrot
Copy link
Contributor

@danrot danrot commented Nov 11, 2013

The portal configuration includes now a field for the resource locator strategy, which can be tree or short

  • test coverage
  • finish the code
  • gather feedback for my changes
Q A
Bug fix? no
New feature? yes
Tests pass? yes
Fixed tickets non
Doc none


<resource-locator>
<strategy>tree</strategy>
</resource-locator>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@chirimoya Do you think that this format is a good decision?

Copy link
Member

Choose a reason for hiding this comment

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

not sure ... do devs who config portals know what a "resource-locator" is?

Copy link
Member

Choose a reason for hiding this comment

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

@drotter @chirimoya i have heard that this is ok (=

wachterjohannes added a commit that referenced this pull request Nov 12, 2013
Added resource locator strategy to portal configuration
@wachterjohannes wachterjohannes merged commit 7693b00 into develop Nov 12, 2013
@wachterjohannes wachterjohannes deleted the feature/portal-configuration branch November 12, 2013 16:38
chirimoya added a commit that referenced this pull request Aug 14, 2014
wachterjohannes added a commit that referenced this pull request Aug 14, 2014
michaelzangerle pushed a commit that referenced this pull request Sep 30, 2014
wachterjohannes added a commit that referenced this pull request Sep 30, 2014
…ield

Do not index the image field as Text
dantleech added a commit that referenced this pull request Sep 30, 2014
michaelzangerle pushed a commit that referenced this pull request Oct 31, 2014
chirimoya added a commit that referenced this pull request Oct 31, 2014
wachterjohannes added a commit that referenced this pull request Oct 31, 2014
dantleech added a commit that referenced this pull request Oct 31, 2014
wachterjohannes added a commit that referenced this pull request Oct 31, 2014
…ield

Do not index the image field as Text
wachterjohannes pushed a commit to wachterjohannes/sulu that referenced this pull request May 8, 2017
alexander-schranz added a commit to alexander-schranz/sulu that referenced this pull request Feb 22, 2024
alexander-schranz pushed a commit to alexander-schranz/sulu that referenced this pull request Nov 25, 2024
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.

3 participants