encrypt/toot

Fork 1

Aggiunti content warning e testo alternativo all'immagine #2

Open

ekardnam wants to merge 2 commits from bida/MastodonMegafono:toot-dev into master

ekardnam commented

2019-02-20 14:12:32 +01:00

First-time contributor

Come dice il titolo, avevo apertu un issue, ma ho sistemato utilizzando un branch secondario.

Con --alt="testo alternativo" o con -a="" si imposta il testo alternativo per persone con disabilità.
Con --warn o -w si imposta il content warning sulle immagini

Come dice il titolo, avevo apertu un issue, ma ho sistemato utilizzando un branch secondario. Con --alt="testo alternativo" o con -a="" si imposta il testo alternativo per persone con disabilità. Con --warn o -w si imposta il content warning sulle immagini

encrypt commented

2019-02-23 22:28:51 +01:00

Owner

Una volta mergiato tutto: non conviene includerlo nel megafono come git submodule? così possiamo sviluppare su un solo repo, evitare di avere due script uguali in due repo diversi e le sbatte per sincronizzare gli aggiornamenti/fix tra uno e l'altro :)

yo, intanto grazie per i commit! Non faccio merge direttamente, ma appena posso ci guardo/testo un attimo, solo per evitare di mergiare roba bash-only (visto che sto coso lo uso in ambienti con solo sh). Una volta mergiato tutto: non conviene includerlo nel megafono come git submodule? così possiamo sviluppare su un solo repo, evitare di avere due script uguali in due repo diversi e le sbatte per sincronizzare gli aggiornamenti/fix tra uno e l'altro :)

ekardnam commented

2019-02-24 17:56:09 +01:00

Author

First-time contributor

yep direi che conviene cosi'

se puoi dare un occhio forse la versione sul master della mia repo è piu' aggiornata, sul mio sistema anche questa della PR non dava problemi, ma su quello dove gira il bot dava un problema con i toot privi d'immagini (tootava con un'immagine mancante) quindi userei la versione piu' nuova e chiuderei direttamente questa PR o la userei solo come discussione :)

ciao!

yep direi che conviene cosi' se puoi dare un occhio forse la versione sul master della mia repo è piu' aggiornata, sul mio sistema anche questa della PR non dava problemi, ma su quello dove gira il bot dava un problema con i toot privi d'immagini (tootava con un'immagine mancante) quindi userei la versione piu' nuova e chiuderei direttamente questa PR o la userei solo come discussione :) ciao!

This pull request can be merged automatically.

You are not authorized to merge this pull request.

View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u toot-dev:bida-toot-dev

git checkout bida-toot-dev

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git checkout master

git merge --no-ff bida-toot-dev

git checkout bida-toot-dev

git rebase master

git checkout master

git merge --ff-only bida-toot-dev

git checkout bida-toot-dev

git rebase master

git checkout master

git merge --no-ff bida-toot-dev

git checkout master

git merge --squash bida-toot-dev

git checkout master

git merge --ff-only bida-toot-dev

git checkout master

git merge bida-toot-dev

git push origin master

No reviewers

No labels

No milestone

No project

No assignees

2 participants

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: encrypt/toot#2

No description provided.