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.

Sign up for GitHub

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

Jump to bottom

Implementacija klase Labele #21

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Implementacija klase Labele #21

wants to merge 2 commits into from

Conversation

Pancake2608
Copy link
Collaborator

@Pancake2608 Pancake2608 commented Mar 28, 2024 •
edited
Loading

  • implementacija __init__ konstruktora
  • [ ]
  • menjanje teksta labele

@Pancake2608 Pancake2608 requested review from Bolsic and Sugaravel March 28, 2024 18:23
@Pancake2608 Pancake2608 self-assigned this Mar 28, 2024
@Pancake2608 Pancake2608 added the enhancement Nova mogućnost ili poboljšanje label Mar 28, 2024
@Pancake2608 Pancake2608 linked an issue Mar 28, 2024 that may be closed by this pull request
Napraviti klasu za labelu #16
Open
@Bolsic
Copy link
Collaborator

Bolsic commented Mar 29, 2024

Iskoristi ovu klasu unutar glavnog koda i pomerite klase u zasebne fajlove pa obrišite fajl "Klase.py"

Bolsic
Bolsic requested changes Mar 29, 2024
View reviewed changes
Copy link
Collaborator

@Bolsic Bolsic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rekao sam sve u komentaru. NIje nešto komplikovana klasa da mora cela da bude svoj PR. Pre nego što je prihvatimo na master granu treba da vidimo dal radi.

Bolsic
Bolsic approved these changes Mar 29, 2024
View reviewed changes
Copy link
Collaborator

@Bolsic Bolsic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okej je samo da se implementira nakon bitnih izmena i implementacije klase EventHandler

@Bolsic Bolsic marked this pull request as ready for review March 29, 2024 12:17
@Bolsic
Copy link
Collaborator

Bolsic commented Mar 29, 2024

Imate konflikte, rešite ih

@Bolsic Bolsic removed the request for review from Sugaravel March 29, 2024 13:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Bolsic Bolsic Bolsic approved these changes

Assignees

@Pancake2608 Pancake2608

Labels
enhancement Nova mogućnost ili poboljšanje
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Napraviti klasu za labelu
2 participants
@Pancake2608 @Bolsic