Skip to content

mvorisek has been really active. Add as maintainer#5

Merged
DarkSide666 merged 1 commit intomasterfrom
add-mvorisek
Feb 11, 2020
Merged

mvorisek has been really active. Add as maintainer#5
DarkSide666 merged 1 commit intomasterfrom
add-mvorisek

Conversation

@romaninsh
Copy link
Copy Markdown
Member

Adding @mvorisek as maintainer to ATK repositories.

Adding @mvorisek as maintainer to ATK repositories.
Copy link
Copy Markdown
Member

@DarkSide666 DarkSide666 left a comment

Choose a reason for hiding this comment

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

welcome

@DarkSide666 DarkSide666 merged commit 3c5e2e2 into master Feb 11, 2020
@DarkSide666 DarkSide666 deleted the add-mvorisek branch February 11, 2020 22:18
@romaninsh romaninsh mentioned this pull request Feb 11, 2020
@mvorisek
Copy link
Copy Markdown
Member

Thank you for inviting me.

My technical focus is primary on:

  • data/DB/core logic
  • clean code/maintainability
  • NOT frontend/css/js outside the obsession for usability 8-)

I will do my best to help with these.

About 2 years ago I started using atk for quick backoffice GUIs. I was always looking for a solution in PHP where I can write the GUI like a code. where I can write view + controller at one place. I would probably give up as the atk data model is still unmature and it simply can not keep with ORMs like Doctrine for larger projects, I also like the data mapper + UoW approach more. But then I have written a very simple but extremely powerfull persistence layer to do all modelling/updates/integrity thru Doctrine and atk is started to rock!

The current largest issue I see is maintainability/refactorability (currently there is almost any - no properties for fields, almost no types in UI) and a little bit too much of freedom (like when UI component can be defined as a seed or as an object). Thank you for the will to work on this together!

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