Skip to content

Conversation

@Voldemortas
Copy link

Andrius Simanaitis

- { property: 'roles', type: 'collection' }
- { property: 'plainPassword', type: 'password' }
- { property: 'homepage', type: 'url' }
entities:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Verta atskirti kodo stiliaus pakeitimus ir funkcionalumo pakeitimą

Plačiau:

@@ -0,0 +1,31 @@
<?php
Copy link
Collaborator

Choose a reason for hiding this comment

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

Nepalikinėk šiukšlių. Ši migracija nieko nedaro

Copy link
Collaborator

Choose a reason for hiding this comment

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

Kodas veikia ➕ buvo atskira pasakita (gal ir be namų darbų) apie Doctrine ➕ vėlai atrašau (jau greitai final atsiskaitymas) – tai už namų darbus balo nemažinu – palieku tavo sąžinei, ar verta pasigilinti

</div>
{% if user.linkedin is not null %}
<div class="mb-2">
<span class="badge badge-primary">LinkedIn</span> <a href="{{ user.linkedin }}"></a>{{ user.linkedin }}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
<span class="badge badge-primary">LinkedIn</span> <a href="{{ user.linkedin }}"></a>{{ user.linkedin }}
<span class="badge badge-primary">LinkedIn</span> <a href="{{ user.linkedin }}">{{ user.linkedin }}</a>

Gal tuo ir skiriasi programuotojai nuo testuotojų... vieni mato kodą, kiti veikiantį kodą

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