Skip to content

refactor: table plugin #2989

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 16 commits into from
May 20, 2025
Merged

refactor: table plugin #2989

merged 16 commits into from
May 20, 2025

Conversation

Jonas-C
Copy link
Contributor

@Jonas-C Jonas-C commented Apr 24, 2025

Brorparten av endringene her er endringer i navngivning og restrukturering av kode. Det eneste som skal ha blitt endret mtp funksjonalitet er i 8db3241.

Det ligger igjen et par TODO-kommentarer man kan ta tak i senere. Tror det er lurt å ta eventuelle forbedringer av matrise-normaliseringen i en ny PR.
Fixes https://github.com/ndlano/issues/issues/4362

@Jonas-C Jonas-C force-pushed the refactor/table-plugin branch from ac33773 to 8dcba5f Compare April 24, 2025 09:56
@Jonas-C Jonas-C force-pushed the refactor/table-plugin branch from 50f6cd7 to ec601eb Compare May 16, 2025 07:50
@Jonas-C Jonas-C marked this pull request as ready for review May 19, 2025 12:18
@Jonas-C Jonas-C requested a review from a team May 19, 2025 12:19
Copy link
Member

@gunnarvelle gunnarvelle left a comment

Choose a reason for hiding this comment

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

Tenker det er fint å få det inn

@Jonas-C Jonas-C merged commit fe26264 into master May 20, 2025
4 checks passed
@Jonas-C Jonas-C deleted the refactor/table-plugin branch May 20, 2025 13:14
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