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

First prototype of adding pageLayout modules #12

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

Sioweb
Copy link

@Sioweb Sioweb commented May 29, 2019

Ho!

ich versuche gerade mit Angular ein einfaches Contao Frontend aufzubauen, dazu habe ich deine API verwendet und etwas erweitert. Die Route /api/page?url=... liefert keine Module aus dem Seitenlayout. Mit der neuen Page-Klasse PageApi werden diese mitgeliefert.

Besteht Interesse, das ganze so weiter zu entwickelt?

PageApi ist im Grunde ein Klon von PageRegular und erweitert einige der Contao\Controller-Methoden, damit diese Arrays statt gerendertem HTML zurückgeben.

Edit: Hier noch mein Ansatz für Angular & Contao: https://github.com/Sioweb/ContaoAngularFrontend

LG
Sascha

@saibotd
Copy link
Contributor

saibotd commented Jun 4, 2019

Sieht gut aus :) Ich sehe es mir die Tage mal genauer an

@Sioweb
Copy link
Author

Sioweb commented Jun 4, 2019

Top, also ich würde hier evt. sogar überlegen ob das so Sinn macht - oder ob wir an der Stelle nicht mit dem Core-Team zusammen überlegen ob die Funktionen die ich "überschrieben" habe nicht generell auch ein Array statt einem fertig geparsten Template etc, zurück geben könnte.

LG
Sascha

@Sioweb
Copy link
Author

Sioweb commented Jun 6, 2019

Ho!

Ich hab den Code noch mal umgebaut. Die Arrays werden jetzt alle über Hooks generiert die Contao native zur Verfügung stellt.

Wrapper-Elemente werden verschachtelt - evt braucht es noch ein Flag um das zu unterbinden.

@saibotd
Copy link
Contributor

saibotd commented Jun 11, 2019

Hi Sascha,

ich würde heute oder morgen gerne ein größeres Update der API bereitstellen, könntest du deinen Request dann nochmal auf den neuesten Stand bringen?
HIntergrund ist, dass wir unsere Vue-Powered Seite letzte Woche online gestellt haben und uns etliche Fehler in der API – z.B. beim Matching von sprachspezifischen URLs – aufgefallen sind ;)

Viele Grüße,
Tobi

@Sioweb
Copy link
Author

Sioweb commented Jun 11, 2019

Klar mach ich :)

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