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.

Sign up for GitHub

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

Jump to bottom

fix(appunti): Correzione e aggiunta appunti Santini #43

Merged
merged 9 commits into from
Aug 17, 2023
Merged

fix(appunti): Correzione e aggiunta appunti Santini #43

merged 9 commits into from
Aug 17, 2023

Conversation

Meht-evaS
Copy link
Contributor

Aggiunta lingua italiana per correggere sillabazione ritorno a capo

Corretti alcuni esempi (sia di buffer overflow che SQL INJECTION)

Aggiornata piccola parte sulla PoS di Etheruem

Corretti errori grammaticali o lessicali

Sostituiti i simboli delle doppie virgolette con gli apici aperti e chiusi

Aggiunto capitolo WebSecurity - Santini

Meht-evaS added 2 commits July 6, 2023 00:34
@Meht-evaS
Correzione Appunti Santini
883cbad 
Aggiunta lingua italiana per correggere sillabazione ritorno a capo

Corretti alcuni esempi (sia di buffer overflow che SQL INJECTION)

Aggiornata piccola parte sulla PoS di Etheruem

Corretti errori grammaticali o lessicali

Sostituiti i simboli delle doppie virgolette con gli apici aperti e chiusi
@Meht-evaS
Aggiunto capitolo WebSecurity - Santini
6b60ad0 
Argomento trattato nell'A.A. 2022/2023 al posto della OS Security
@ncvescera ncvescera changed the title Correzione e aggiunta appunti Santini fix(appunti): Correzione e aggiunta appunti Santini Jul 6, 2023
@ncvescera ncvescera added the riscrittura Riscrittura o completamento di materiale incompleto label Jul 6, 2023
ncvescera
ncvescera requested changes Jul 6, 2023
View reviewed changes
Copy link
Member

@ncvescera ncvescera left a comment

Choose a reason for hiding this comment

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

Ho letto e controllato le varie modifiche.
In linea di massima:

  • Ottimo l'aver convertito le 'virgolette' in vere virgolette
  • I vari typo fix
  • Le nuove immagini di dimensione adeguata
  • Le nuove aggiunte sulla Web Security

C'è qualcosa da ricontrollare:

  • Eliminare tutti i \vspace (chiediamo anche il parere degli altri)
  • Hai mancato alcune 'virgolette'
  • Nella nuova parte ci sono alcuni comandi e file che sono scritti come testo normale e non come \verb|| o simili

@ncvescera
Copy link
Member

@CristianCosci @F-a-b-r-i-z-i-o @NicoloPosta @TommasoRomani Siamo di fronte ad un dilemma, @Meht-evaS ha apportato delle modifiche agli appunti di Santini e Bistarelli (Cybersecurity) ed ha aggiunto dei \vspace nel codice LaTex per andare ad aggiungere o togliere spazio tra i titoli ed il testo nornale (non solo, ma molto spesso).

Possiamo qui vederne alcuni effetti

image

image

image

Io personalmente non penso vadano utilizzati dato che vanno a sbilanciare molto l'equilibrio che c'è nel PDF. Le distanze che LaTex imposta di default penso non vadano cambiate o meglio, vanno impostate all'inizio in modo da avere una continuità e coerenza in tutto il documento.
Preferisco che alcuni parti del testo vadano a pagina nuova invece che metterle tutte nella stessa pagina andando a modificare le varie distanze.

Voi cosa ne pensate ? Li teniamo o li togliamo ?

@CristianCosci
Copy link
Collaborator

Anche io penso che non ci sia la necessità di utilizzare i vari vspace in quanto crea asimmetrie con il resto del pdf e anche in generale con l'approccio di tutti i vari appunti. Tendenzialmente li utilizzerei solo per la gestione particolare di alcune foto o di un distacco all'interno di testo dello stesso paragrafo/sezione per enfatizzarlo.

@ncvescera
Copy link
Member

Tendenzialmente li utilizzerei solo per la gestione particolare di alcune foto o di un distacco all'interno di testo dello stesso paragrafo/sezione per enfatizzarlo.

A questo non avevo pensato. Direi può andare bene così 🚀
@Meht-evaS Vediamo se arriveranno altri pareri, sennò puoi procedere come suggerisce @CristianCosci

@Meht-evaS
Correzioni aggiuntive appunti Santini
eb64ed5 
Effettuate le modifiche fatte presenti nella Pull Request:

- Eliminati quasi tutti i vspace
- Aggiunte virgolette mancanti
- Aggiunta stile \verb|| a nomi funzioni
- Migliorata definizione `Expires` cookie
@Meht-evaS
Copy link
Contributor Author

Ho applicato tutte le correzioni consigliate. Ho solo lasciato un paio di \vspace{} attorno a una foto in capitolo6_threads e capitolo9_xss perchè a toglierli si sformattano troppe cose. Se pensate che è meglio togliere anche quelli fatemelo sapere.
@ncvescera @CristianCosci

ncvescera
ncvescera requested changes Jul 7, 2023
View reviewed changes
Copy link
Member

@ncvescera ncvescera left a comment

Choose a reason for hiding this comment

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

I restanti vspace possiamo lascialri.
C'è un piccolo errore che ho trovato, ma per il resto ottimo 👍🏻

@Meht-evaS
Correzione aggiuntiva appunti Santini
3b6d577 
Eliminato un segno di punteggiatura ridondante.
ncvescera
ncvescera approved these changes Jul 10, 2023
View reviewed changes
Copy link
Member

@ncvescera ncvescera left a comment

Choose a reason for hiding this comment

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

👍🏻

ncvescera
ncvescera requested changes Jul 25, 2023
View reviewed changes
Copy link
Member

@ncvescera ncvescera left a comment

Choose a reason for hiding this comment

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

Questa PR non è pronta per il Merge, va rimosso il file del font e ripristiano il font originale

@ncvescera
fix: devcontainer & main.tex
4aed3da 
Sistemato un problema che impediva la corretta build del progetto (il
Dockerfile era vecchio).
Aggiornato .gitignore con il template per LaTex.
Ripristinato path del font per le emoji originale.
@ncvescera
Aggiornato PDF
0abec5d
@ncvescera
fix: frontmatter
2eec9a8 
Modificato il frontmatter mandando a capo il nome dell'esame dalla
parola "Appunti" e rendendolo in italico.
@ncvescera
Aggiornato PDF
b57906a
@ncvescera ncvescera added the ready La Pull Request è pronta per il merge label Aug 9, 2023
@ncvescera
Copy link
Member

Prima di effetutare il merge di questa risolviamo la #59 così abbiamo la release funzionante.

@ncvescera ncvescera merged commit c3e1e13 into Typing-Monkeys:master Aug 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ncvescera ncvescera ncvescera approved these changes

@NicoloPosta NicoloPosta NicoloPosta approved these changes

@CristianCosci CristianCosci Awaiting requested review from CristianCosci

@TommasoRomani TommasoRomani Awaiting requested review from TommasoRomani

@F-a-b-r-i-z-i-o F-a-b-r-i-z-i-o Awaiting requested review from F-a-b-r-i-z-i-o

Assignees

@CristianCosci CristianCosci

@TommasoRomani TommasoRomani

@NicoloPosta NicoloPosta

@F-a-b-r-i-z-i-o F-a-b-r-i-z-i-o

Labels
ready La Pull Request è pronta per il merge riscrittura Riscrittura o completamento di materiale incompleto
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@Meht-evaS @ncvescera @CristianCosci @NicoloPosta @F-a-b-r-i-z-i-o @TommasoRomani