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/sling12 #682

Closed
wants to merge 59 commits into from
Closed

Update/sling12 #682

wants to merge 59 commits into from

Conversation

cmrockwell
Copy link
Collaborator

Addressing issues

  1. Repoinit ACL's need restart #677 Repoinit was setting ACL's for paths that did not exist before package install resulting in errors
  2. New tenant can't log in on first ever Peregrine launch #661 fixing repoinit resolves new tenant issue on first ever peregrine launch
  3. update feature model for develop-sling12 to latest #644 updates peregrine starter to latest sling starter
  4. Exceptions on develop-sling12 peregrine startup #568 resolved startup errors
  5. Startup exception in Sling12 #604 resolved startup errors
  6. Composum broken in develop-sling12 #569 resolved startup errors
  7. Page editing error (pagerenderserver) #621 resolved pagerender/server page editing error
  8. Container for pagerenderserver #620 resolved pagerender/server container
  9. Server Side HTL (pagerenderer/server) #618 resolved pagerender/server rendering errors

…h common-lang dependency. Also frontend-maven-plugin (node) was failing, but not sure what it does for server/ui.apps. Commenting out for now, also the module to build, install and deploy to Sling
Fix NPE due to incorrect the init method local variable 'resource' overriding instance property
…mple vue site can have new page without errors. Added TODO comments about nav and footer for these sites
…nderserver/components. Adjusted PageRenderServerConstants.java and /content/pagerenderserver/ paths accordingly
…tion (AdaptTo) for per:Page and per:Asset types; and their ability to get the correct per:Site resource as implemented in PerBaseImpl.java
…. Refactored Helper.java to allow siteCSS to exist under per:Site/jcr:content
…e pagerenderer/server module, it may be useful to continue support the npm build processes. this commit updates a few deps to keep updated with the other modules. may be necessary to mvn clean -PcleanNodeModules or reinstall via npm i.
…rning a list of resources coming either from the template or the page content.
…elpers alike. Refactored PageHelper Use class to extend BaseHelper
…ed PageHelper to use getters instead of direct access
…riptHelper is null, then it will try to get the modelFactory from the bindings, which is added by the Adaption Jtest adds it. Use classes have bindings such as resource. Getting the resource from the request was problematic for testing, it is easier to test if PageMaerg.java uses the resource reference from the bindings, and not the reference attached to the request
…d to use resource from the bindings and not a reference through the request
…st whether getMergedResources properly merges template content with page content for HTL rendering
…iners in template with page content containers
…n such that refreshEditor is called and rendering is refreshed showing added components
…ows getting parent content from tenant root per:Site/jcr:content
…pages. Tests that new pages they have containers for authoring and resources inherited from the template
…t merges children and properties for template and page resources
… that they exist when setting their ACL's. Added Create folder directives for folders that might not exist yet in RepositoryInitializer-peregrine.config
…IT, addded running IT profile to install-peregrine.sh (for the docker build and integration test)
@cmrockwell cmrockwell closed this Dec 15, 2020
@flx-sta flx-sta deleted the update/sling12 branch March 24, 2021 11:59
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.

2 participants