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

Add ussues labels and filtering #268

Merged
merged 5 commits into from May 3, 2023
Merged

Add ussues labels and filtering #268

merged 5 commits into from May 3, 2023

Conversation

ya-pekatoros
Copy link
Contributor

Реализовано отображение лэйблов и модель лэйблов для issues. В настоящий момент проблема в том, что при мультивыборе создаются дублирующие строки в таблице в интерфейсе. Ниже демонстрация.

https://www.youtube.com/watch?v=eBiltDQ3dBM

@fey
Copy link
Collaborator

fey commented Apr 7, 2023

А какие запросы генерируются в базу?
По идее должна быть выборка, усовно через many to many связь и поиском записей у которых есть все множество лейблов и DISTINCT по id ишшуса

@ya-pekatoros
Copy link
Contributor Author

ya-pekatoros commented Apr 7, 2023

Refer

Все так. Сама страница изначально отрисовывается корректно и при фильтре по одному лэйблу тоже. Но при мультивыборе происходит какое-то дублирование, но проблема где-то на стороне миксинов и прочих расширений.... я так и не смог понять где, там так все лихо накручено =)

@fey
Copy link
Collaborator

fey commented Apr 7, 2023

Так а если поудалять лишнее?

@ya-pekatoros
Copy link
Contributor Author

ya-pekatoros commented Apr 7, 2023

Так а если поудалять лишнее?

Их не должно быть, но они есть. Может быть у вас есть идеи в чем ошибка? У меня, к сожалению, они кончились.

@fey
Copy link
Collaborator

fey commented Apr 7, 2023

Нет, у меня нет идей, я не эксперт по джанге :D
Я предлагаю подебажить и посмотреть, какие запросы выполняются. Чтобы понять, почему данные такие.

@ya-pekatoros
Copy link
Contributor Author

Нет, у меня нет идей, я не эксперт по джанге :D Я предлагаю подебажить и посмотреть, какие запросы выполняются. Чтобы понять, почему данные такие.

Я пробовал, но мое терпение истощилось. Потому и опубликовал в таком виде. Быть может кто-то сразу поймет (может быть есть те, кто знаком с кодовой базой проекта).

@ssssank
Copy link
Contributor

ssssank commented Apr 11, 2023

@emp7yhead без тебя никак :)

@emp7yhead
Copy link
Contributor

@ya-pekatoros почини ошибки линтера, пожалуйста

@ssssank
Copy link
Contributor

ssssank commented Apr 20, 2023

@ya-pekatoros как успехи? нужна ли помощь?

@ya-pekatoros
Copy link
Contributor Author

@ya-pekatoros как успехи? нужна ли помощь?

так я ведь и создал PR с целью получить помощь в возникшем баге, линтер нас тут не спасет. Предлагаю сначала устранить баг, потом я все поправлю и тесты сделаю.

@ya-pekatoros
Copy link
Contributor Author

@ya-pekatoros почини ошибки линтера, пожалуйста

Это ведь не поможет в озвученной выше проблеме.

@ssssank
Copy link
Contributor

ssssank commented Apr 21, 2023

Попробуй задеплоить твой код куда-нибудь, на Railway, например. Чтобы мы смогли "пощупать" руками то, что происходит

@fey
Copy link
Collaborator

fey commented Apr 21, 2023

@ya-pekatoros даже с багой мы можем задеплоить текущую версию и потом поправить.
А если линтер жалуется на ошибки, то смержив, придется их все равно править.

@ya-pekatoros
Copy link
Contributor Author

ya-pekatoros commented Apr 23, 2023

Попробуй задеплоить твой код куда-нибудь, на Railway, например. Чтобы мы смогли "пощупать" руками то, что происходит

@fey @ssssank
http://34.125.196.152:8000/issues?contribution_labels=bug.help+wanted.good+first+issue

Вот демонстранция бага =) Линтеры нет смысла трогать, потому что вся суть PR в этой страничке, которая не работает, как ожидается.

Машина слабенькая хостит, так что наберитесь немного терпения чтобы загрузилась)

@ya-pekatoros
Copy link
Contributor Author

@fey @ssssank Посмотри пожалуйста. Вроде бы теперь все ок.

@ssssank
Copy link
Contributor

ssssank commented Apr 24, 2023

@ya-pekatoros а ссылочка не открывается :(

@ya-pekatoros
Copy link
Contributor Author

@ya-pekatoros а ссылочка не открывается :(

запустил, сейчас обновлю только

@ya-pekatoros
Copy link
Contributor Author

@ya-pekatoros а ссылочка не открывается :(

посмотрите, сейчас все ок, все лэйблы в т.ч. в таблице кликабельны, все работает вроде бы

@fey
Copy link
Collaborator

fey commented Apr 24, 2023

@ya-pekatoros вроде бы норм. Я предлагаю мержить, тк щас страницей не особо пользуются и будем дальше править.
Я предлагаю созвониться и расскажу, как все это вижу и куда двигаться.

@ya-pekatoros
Copy link
Contributor Author

@ya-pekatoros вроде бы норм. Я предлагаю мержить, тк щас страницей не особо пользуются и будем дальше править. Я предлагаю созвониться и расскажу, как все это вижу и куда двигаться.

ок, я пока в ближайшие пару дней в любое время свободен

@fey fey merged commit 8067a19 into Hexlet:main May 3, 2023
1 check passed
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.

None yet

4 participants