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

blob #244

Merged
merged 9 commits into from
Apr 16, 2024
Merged

blob #244

merged 9 commits into from
Apr 16, 2024

Conversation

mamico
Copy link
Contributor

@mamico mamico commented Feb 15, 2024

No description provided.

@mamico mamico requested a review from cekk February 15, 2024 10:28
@coveralls
Copy link

coveralls commented Feb 15, 2024

Pull Request Test Coverage Report for Build 8676363008

Details

  • 10 of 10 (100.0%) changed or added relevant lines in 5 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 69.449%

Totals Coverage Status
Change from base Build 8676361292: 0.0%
Covered Lines: 3303
Relevant Lines: 4756

💛 - Coveralls

@mamico mamico marked this pull request as ready for review February 24, 2024 06:56
@mamico
Copy link
Contributor Author

mamico commented Apr 13, 2024

@cekk @luca-bellenghi

@luca-bellenghi
Copy link
Member

@mamico si, decisamente serve.
Ho un unico dubbio per metterla su ora: i contenuti funzionano anche senza l'upgrade step? immagino di no. Il problema è che a fronte di comuni molto piccoli, dove non vedo problemi a procedere, ci sono anche comuni grandi in cui può essere problematico se le immagini non vanno. è anche vero, e questo secondo me sarà la soluzione, che alla fine ci sono solo le immagini delle persone. se guardo il mapping, documento personale, messaggio e ricevuta non li usiamo.

Nel caso, farei il merge subito dopo aver pinnato la versione precedente di design.plone.contentypes così domani la provo a mano forzando un paio di comuni.

@mamico
Copy link
Contributor Author

mamico commented Apr 16, 2024

Dalle mie prove funziona tutto anche senza upgrade step

@mamico
Copy link
Contributor Author

mamico commented Apr 16, 2024

Ok comunque se la provi a campione domani.

@luca-bellenghi luca-bellenghi merged commit 0f0cd3e into main Apr 16, 2024
9 checks passed
@luca-bellenghi
Copy link
Member

Ok comunque se la provi a campione domani.

perfetto. c'è già la 6.2.4 con la tua PR. non è rilasciata in iocomune-backend.
La provo su un paio di comuni "sensati" e poi la introduco.

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