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

Added the "Cloud Italia" platform, deleted the "DevOps" one. #405

Merged
merged 26 commits into from
Oct 3, 2019
Merged

Added the "Cloud Italia" platform, deleted the "DevOps" one. #405

merged 26 commits into from
Oct 3, 2019

Conversation

luciopileggi
Copy link
Member

Added the "Cloud Italia" platform: italian and english (#345) pages plus a 800x400 PNG logo. Deleted devops.md platform page. @paolo-de-rosa

…lus a 800x400 PNG logo. Deleted devops.md platform page. @paolo-de-rosa
Copy link
Member

@libremente libremente left a comment

Choose a reason for hiding this comment

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

Just a couple of remarks @luciopileggi

_platforms/en/cloud.md Outdated Show resolved Hide resolved
_platforms/en/cloud.md Outdated Show resolved Hide resolved
@sebbalex
Copy link
Member

CI is failing because of #406

luciopileggi and others added 2 commits September 24, 2019 15:15
changed "Three-Year plan" link to point a translated page

Co-Authored-By: libremente <surf@libremente.eu>
Co-Authored-By: libremente <surf@libremente.eu>
luciopileggi and others added 2 commits September 24, 2019 18:13
Co-Authored-By: Alessandro Sebastiani <sebbalex@users.noreply.github.com>
@libremente
Copy link
Member

There is a visual issue:
Schermata 2019-09-30 alle 11 07 57

Copy link
Member

@libremente libremente left a comment

Choose a reason for hiding this comment

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

Small typos

_platforms/it/cloud.md Outdated Show resolved Hide resolved
_platforms/it/cloud.md Outdated Show resolved Hide resolved
_platforms/en/cloud.md Outdated Show resolved Hide resolved
_platforms/en/cloud.md Outdated Show resolved Hide resolved
_platforms/en/cloud.md Outdated Show resolved Hide resolved
luciopileggi and others added 7 commits September 30, 2019 12:48
Co-Authored-By: libremente <surf@libremente.eu>
Co-Authored-By: libremente <surf@libremente.eu>
Co-Authored-By: libremente <surf@libremente.eu>
Co-Authored-By: libremente <surf@libremente.eu>
Co-Authored-By: libremente <surf@libremente.eu>
@libremente
Copy link
Member

@luciopileggi ho notato che il logo dice Il Cloud della PA ma in cloud.italia.it il titolo è Il cloud della PA con la c minuscola.

_platforms/en/cloud.md Outdated Show resolved Hide resolved
_platforms/en/cloud.md Outdated Show resolved Hide resolved
_platforms/en/cloud.md Outdated Show resolved Hide resolved
_platforms/en/cloud.md Outdated Show resolved Hide resolved
_platforms/en/cloud.md Outdated Show resolved Hide resolved
_platforms/en/cloud.md Outdated Show resolved Hide resolved
_platforms/en/cloud.md Outdated Show resolved Hide resolved
luciopileggi and others added 8 commits October 1, 2019 15:37
Co-Authored-By: libremente <surf@libremente.eu>
Co-Authored-By: libremente <surf@libremente.eu>
Co-Authored-By: libremente <surf@libremente.eu>
Co-Authored-By: libremente <surf@libremente.eu>
Co-Authored-By: libremente <surf@libremente.eu>
Co-Authored-By: libremente <surf@libremente.eu>
@luciopileggi
Copy link
Member Author

There is a visual issue:
Schermata 2019-09-30 alle 11 07 57

Resolved!

_platforms/en/cloud.md Show resolved Hide resolved
_platforms/en/cloud.md Outdated Show resolved Hide resolved
Co-Authored-By: libremente <surf@libremente.eu>
Copy link
Member

@libremente libremente left a comment

Choose a reason for hiding this comment

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

LGTM! 👍

@libremente libremente merged commit 57f314d into italia:master Oct 3, 2019
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