-
Notifications
You must be signed in to change notification settings - Fork 38
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
feat(communities): Angular Austria #35
base: master
Are you sure you want to change the base?
Conversation
"twitter": "https://twitter.com/michael_hladky" | ||
"twitter": "https://twitter.com/michael_hladky", | ||
"webs": ["https://www.linkedin.com/in/michael-hladky-519340148/"] | ||
}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Split the PR into two PRs due to 1PR add information about one community and the second PR is a new Community
"Angular Austria": { | ||
"id": 87, | ||
"name": "Angular Austria", | ||
"position": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Change the position because Aungular-Austria and Angular-Viena have the same latitude/longitude. The communities must to have different latitude/longitude
"lng": 14.510690, | ||
"lat": 47.739257 | ||
}, | ||
"city": "Wien, Österreich", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- We need a shield in each community
} | ||
] | ||
}, | ||
"Angular Austria": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- is AngularAustria is a local-community which do events about Angular? Is this community related with Angular-Vienna ?
"twitter": "https://twitter.com/michael_hladky", | ||
"webs": ["https://www.linkedin.com/in/michael-hladky-519340148/"] | ||
}, | ||
{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Could you move this information to the end of file (only for organization).
Hey @karol-depka, can you resolve the requested changes? |
I think we should close the PR, and redo the changes ourselves in new PRs. @karol-depka are you there? |
Hi, @tonivj5 . I'm sorry, I didn't have time. Let me try to do it this week? Apologies for the delay. Saludos! |
No description provided.