сабж

  • cocuckaOP
    link
    fedilink
    arrow-up
    0
    ·
    4 years ago

    Возможно @crypt между делом залил версию без последних фиксов. У нас тут ковбойский стиль обновления прода.

    • crypt
      link
      fedilink
      arrow-up
      0
      ·
      4 years ago

      да, ковбойский, не могу с этим спорить. ща проверю, что я залил.

    • Harald
      link
      fedilink
      arrow-up
      0
      ·
      4 years ago

      а как же тестовый сервер, стейджинг и прочие бест практис, девопс и high availability? ^)

      • cocuckaOP
        link
        fedilink
        arrow-up
        0
        ·
        4 years ago

        Мы по этой книге работаем:

        • crypt
          link
          fedilink
          arrow-up
          0
          ·
          4 years ago

          вообще этот новый вид трекера надо совсем выпилить. ты согласен?

            • crypt
              link
              fedilink
              arrow-up
              0
              ·
              4 years ago

              я ща просто обновлю всем сеттинги на settings[‘oldTracker’] = true. или false. я так уже делал. надо только не перепутать. а нет.

            • crypt
              link
              fedilink
              arrow-up
              0
              ·
              4 years ago

              погоди! а что там за новые фиксы? я помню, что менял, чтобы независимо от настроек трекер всегда был старый. я просто не решился трогать скала-код. а ты что-то изменил в этом?

              • cocuckaOP
                link
                fedilink
                arrow-up
                0
                ·
                4 years ago

                Ты убрал галочку «старый вид трекера» со страницы, но код который сохраняет ее значение не поправил. В итоге, если менять настройки в профиле, то при сохранении запишется значение false. Обратно уже не поменять, галочки то нет

                • crypt
                  link
                  fedilink
                  arrow-up
                  0
                  ·
                  4 years ago

                  да, но сайт должен всегда показывать старый вид трекера. так какая разница, что выставлено?

                  • cocuckaOP
                    link
                    fedilink
                    arrow-up
                    0
                    ·
                    4 years ago

                    Ну в ru/org/linux/tracker/TrackerController.scala:123 проверка осталась.

                    Я почищу, мб даже сегодя.

                    • crypt
                      link
                      fedilink
                      arrow-up
                      0
                      ·
                      4 years ago

                      я сам попробую сделать. пили что-то крупное, а мелочи оставь мне.

                    • crypt
                      link
                      fedilink
                      arrow-up
                      0
                      ·
                      4 years ago

                      я бы тебя оч. попросил ворнинги при комиляции почистить. это требует применения новых методов и это для меня сложно. а с треккером разберусь.

                      .scala:76: warning: object ElasticsearchClientUri in package elastic4s is deprecated (since 6.3.3): Use ElasticNodeEndpoint
                      
                    • crypt
                      link
                      fedilink
                      arrow-up
                      0
                      ·
                      4 years ago

                      Я почищу, мб даже сегодя.

                      убрал и выкатил.

                    • crypt
                      link
                      fedilink
                      arrow-up
                      0
                      ·
                      4 years ago

                      было бы здорово, если бы ты достал из истории гита *java файлы, которые были заменены на *scala и интегрировал их обратно. я сталкиваюсь с тем, что у @maxcom scala код идет вперемешку с версткой html. это прям проблема для исправлений. jsp меня очень выручает.

    • crypt
      link
      fedilink
      arrow-up
      0
      ·
      4 years ago

      похоже, так и есть. сейчас залью со всеми фиксами. сорри.