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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

[META] 馃悰 Bug: some fix from integration test phase for 4.8.0+build_128 #6436

Open
9 of 20 tasks
floreal15 opened this issue Apr 15, 2024 · 1 comment
Open
9 of 20 tasks
Assignees
Labels
backend issue related to backend developpement /apps bug Issues that had not yet been identified at the time of development delivery. frontend issue related to frontend development / ux / ui / apps meta The issue is a meta-ticket

Comments

@floreal15
Copy link
Collaborator

floreal15 commented Apr 15, 2024

Tracim version

4.8.0 build_128

Issue description

  • Fix path to tracimcli for mail summary in docker

  • Sed line in common.sh refer to nothing

    • Additional information: line in common.sh doesn't seems to be used anymore
    sed -i "s|<PATH TO development.ini>|/etc/tracim/development.ini|g" /etc/tracim/send_summary_mails.sh
  • Typo in a Spanish translation聽#6439

    • Additional information: It generate stack trace in server log when sending mail summary
    #: tracim_backend/templates/mail/summary_body_html.mak:24
    msgid "{author} mentioned you in"
    msgstr "{autor} le mencion贸 en"
    
  • Mail subject should correspond to the mail send ( hourly, daily or weekly)

    • to fix: in periodic.py::_send_mail()
      • translation
  • Reaction enable in advanced dashboard

    • Additional information: error 400 - Validation error of input data" when used
    • to fix : bring back the old mechanism to decide whether or not display the emoji (props showReaction)
  • Notification type selection isn't taken into account if we click on the right side ( on or after the ? )

    • to fix : move the handler of the click on all the line instead of the left side
  • In Spanish the notification management title overlap the selector

    • To fix: remove the white-space: nowrap;
  • Feat : UX - the confirmation popup for the true delete has a bad UX

    • Additional information: the confirmation popup for the true delete ( in french) isn't center
    • the hole popup is unbalanced ( button aren't center, title on two line )
  • UX - the help ( ? button) for the notification choice start behind the list before coming before

    • Additional information: blinking of the help component, not user friendly
    • To test : force a z-index ; deactivate the appearing animation
  • Email summary log didn't indicate if the error came from an hourly, daily or weekly

    • Additional information: script evolution or make 3 log file
  • UX - Card color in the logbook

    • Additional information: The mechanism from the kanban to automatically set the text color should be used in the logbook too
  • Space in double

    • Additional information: In the side bar some space can be duplicate, if we click on one the other appear as active too.
    • they disappear on reload
  • In split screen, in recent activities video player go outside off the border on the right

    • Additional information: the video player is in the iframe
    • in a resolution of 900~1000px
  • UX - emoji in the Open field go outside of the border if their is to many

    • Additional information: the emojis start by reducing the title before exiting the card if their is to many
    • their is a white-space: nowrap; on the element
    • maybe a solution is limiting the number a character
  • UX - popup for creating an office doc got a css problem with Chrome, the option exceed

  • televersement popup size isn't large and change according to the language

  • translation - the translation for the summary notification ( hourly, daily, weekly ) isn't done in most language

  • translation - the translation for the summary notification help isn't done in most language

  • translation - the translation for the logbook isn't done in most language

    • Additional information: the element aren't translate and appear in french "Cr茅er un journal" "Nouveau journal", "Pas encore d'茅v猫nement" and "Cr茅er un 茅v猫nement"

How to reproduce

Screenshots

No response

What browsers are you seeing the problem on?

No response

What devices are you seeing the problem on?

No response

Related PR

#6437

@floreal15 floreal15 added to sort need first level analyse and release association bug Issues that had not yet been identified at the time of development delivery. labels Apr 15, 2024
@floreal15 floreal15 added this to In progress in Tracim v4.9.0 Apr 16, 2024
@floreal15 floreal15 added frontend issue related to frontend development / ux / ui / apps backend issue related to backend developpement /apps meta The issue is a meta-ticket and removed to sort need first level analyse and release association labels Apr 16, 2024
@floreal15 floreal15 self-assigned this Apr 16, 2024
@Skylsmoi Skylsmoi changed the title [META] 馃悰 Bug: some fix [META] 馃悰 Bug: some fix from integration test phase for 4.8.0+build_128 Apr 17, 2024
@Skylsmoi Skylsmoi moved this from In progress to To do in Tracim v4.9.0 Apr 18, 2024
@Skylsmoi
Copy link
Collaborator

regressions and blocking issues have been fixed in build_129

@Millefeuille42 Millefeuille42 added this to To do in Tracim v4.10.0 Apr 25, 2024
@Millefeuille42 Millefeuille42 removed this from To do in Tracim v4.9.0 Apr 25, 2024
@Skylsmoi Skylsmoi moved this from To do to Issue to review in Tracim v4.10.0 May 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend issue related to backend developpement /apps bug Issues that had not yet been identified at the time of development delivery. frontend issue related to frontend development / ux / ui / apps meta The issue is a meta-ticket
Projects
Tracim v4.10.0
Issue to review
Development

No branches or pull requests

2 participants