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

KAS-4678: Update van rdfa-editor package naar meest recente versie #2141

Merged
merged 37 commits into from
Nov 26, 2024

Conversation

brenner-company
Copy link
Contributor

@brenner-company brenner-company commented Jun 18, 2024

Nog niet mergen! De testen moeten eerst slagen, en we releasen dit best als standalone upgrade, evt. samen met de Ember upgrade.

Tickets binnen Jira:

Andere PRs die samen hiermee moeten gemerged worden:
kanselarij-vlaanderen/decision-report-generation-service#22
kanselarij-vlaanderen/minutes-report-generation-service#16

brenner-company and others added 26 commits May 15, 2024 13:00
…nd-kaleidos into DES/KAS-4678-rdfa-editor-package-update
…editor-package-update

# Conflicts:
#	package-lock.json
#	package.json
…nd-kaleidos into DES/KAS-4678-rdfa-editor-package-update
@ValenberghsSven
Copy link
Contributor

ValenberghsSven commented Nov 5, 2024

Er zijn nog enkele issues naar boven gekomen:
wij gebruikten custom html "id" properties om achteraf vervangingen te kunnen doen (of automatische overschrijvingen tegen te gaan)
Zoals de lijst aanwezigen en afwezigen (niet overschrijven bij update), de secretaris (vervangen wanneer nodig) en de info over de volgende vergadering (niet overschrijven) in notulen.
Alsook voor styling toe te passen.

Onze id's worden niet meer aanvaard in de html en worden gestript. Al onze logica errond is dus stuk.
De logica met onze custom "section" werkt wel nog dus ik ben al wat aan het vervangen om die sections te gebruiken.
Dit gebruikten we om agendapunten en mededelingen overschrijven tegen te gaan (tenzij expliciet "agendapunten updaten" gebruikt wordt)

Daarnaast ook een probleem met tabellen. Vroeger waren de kolommen een vaste breedte (50/50 van de tabel).
Nu klopt dat niet meer
image

En die lijkt nu ook dynamisch te zijn > als je begint te typen in bvb de afwezigen tabel dan begint de middenlijn te verschuiven.
image

Je kan die ook manueel verschuiven:
image

Daarna blijft die even breed. Maar we zouden dus een start breedte moeten kunnen instellen.

@ValenberghsSven
Copy link
Contributor

ValenberghsSven commented Nov 6, 2024

check out

It solves the issue of the id's no longer working. (requires minutes-report kanselarij-vlaanderen/minutes-report-generation-service#17)
! It does not not solve the table issue with uneven and shifting column widths. !

@brenner-company
Copy link
Contributor Author

brenner-company commented Nov 11, 2024

Daarnaast ook een probleem met tabellen. Vroeger waren de kolommen een vaste breedte (50/50 van de tabel). Nu klopt dat niet meer.

@ValenberghsSven Door een data-colwidth te definiëren op de (nodige) cellen krijgen ze nu wel al de correcte initiële waarde. De kolommen zijn nog wel steeds qua breedte verstelbaar, maar dat zal vermoedelijk geen issue zijn?

…678-rdfa-editor-package-update-sections

# Conflicts:
#	app/controllers/agenda/minutes.js
…-editor-package-update-sections

KAS-4678 use sections everywhere for custom rdfa editor parts
@ValenberghsSven
Copy link
Contributor

Daarnaast ook een probleem met tabellen. Vroeger waren de kolommen een vaste breedte (50/50 van de tabel). Nu klopt dat niet meer.

@ValenberghsSven Door een data-colwidth te definiëren op de (nodige) cellen krijgen ze nu wel al de correcte initiële waarde. De kolommen zijn nog wel steeds qua breedte verstelbaar, maar dat zal vermoedelijk geen issue zijn?

Ziet er goed uit nu @brenner-company

FYI: Ik heb deze branch en andere rond ember allemaal in 1 branch getrokken om te kijken of ik al naar ember v5 kon.
#2255

…nd-kaleidos into DES/KAS-4678-rdfa-editor-package-update
…nd-kaleidos into DES/KAS-4678-rdfa-editor-package-update
@ValenberghsSven ValenberghsSven merged commit 9b290fd into development Nov 26, 2024
1 of 2 checks passed
@ValenberghsSven ValenberghsSven deleted the DES/KAS-4678-rdfa-editor-package-update branch November 26, 2024 11:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants