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

68-rola-autora #361

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

68-rola-autora #361

wants to merge 13 commits into from

Conversation

chibika
Copy link
Contributor

@chibika chibika commented Feb 20, 2016

Jeszcze nie zaczęłam, czy mógłby mi Pan jeszcze raz wyjaśnić jak rola autora ma być przyporządkowana do powiązania autora z danym artykułem? Teraz zrobiłam ją po prostu jako jeden z atrybutów zgłoszenia, ale to chyba ma być jakoś inaczej?
#68

@chibika chibika changed the title wersja 0.1 68-rola-autora Feb 20, 2016
@apohllo
Copy link
Owner

apohllo commented Feb 23, 2016

To ma być atrybut w Authroship

@apohllo
Copy link
Owner

apohllo commented Feb 26, 2016

@chibika Czy potrzebne są jeszcze jakieś wyjaśnienia? Bo zadanie wymaga jeszcze sporo pracy.

@chibika
Copy link
Contributor Author

chibika commented Feb 26, 2016

Czy tak to ma wyglądać w bazie danych? Poza tym muszę dodać pole i tooltip w formularzu i dodać wyświetlanie tego pola w widokach - czy coś jeszcze (poza testami oczywiście)? Jaka ma być treść tego tooltipa?
edit: teraz widzę że jest źle, zaraz poprawię

@@ -1,6 +1,7 @@
class Authorship < ActiveRecord::Base
validates :person, presence: true, uniqueness: { scope: :submission_id }

validates :author_role, presence: true
Copy link
Owner

Choose a reason for hiding this comment

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

wystarczyłoby role

@apohllo
Copy link
Owner

apohllo commented Feb 26, 2016

To pole powinno pojawić się w formularzu dodawania autora. W widoku zgłoszeń jest guzik - dodaj autora i po jego kliknięciu powinna być możliwość określenia "roli". Trzeba zmodyfikować widok app/views/authorships/_form.html.haml oraz dodać permit(....,:role) na dole w authorships_controller

@chibika
Copy link
Contributor Author

chibika commented Mar 3, 2016

nie mogę skończyć zadania bo od kilku dni mam ten sam problem z c9.io, aplikacja nie chce mi się ładować i co chwila mam taki komunikat:
error

przez to nie widzę, co robię i gdzie mam błędy :(

@apohllo
Copy link
Owner

apohllo commented Mar 3, 2016

Powinno być http zamiast https - przynajmniej tyle mogę wyczytać z komunikatu...

@chibika
Copy link
Contributor Author

chibika commented Mar 3, 2016

teraz widzę że muszę poprawić testy, no i update'ować

@apohllo
Copy link
Owner

apohllo commented Mar 3, 2016

Prościej będzie usunąć wymóg dotyczący roli autora - to się zrobi w osobnym zadaniu.

@@ -1,6 +1,6 @@
class Authorship < ActiveRecord::Base
validates :person, presence: true, uniqueness: { scope: :submission_id }

validates :role, presence: true
Copy link
Owner

Choose a reason for hiding this comment

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

Proszę to usunąć - to rozwiąże problem testów.

@apohllo
Copy link
Owner

apohllo commented Mar 3, 2016

Nie wymagam dodatkowego testu do tego zadania. Po updacie względme mastera będę mergował.

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.

2 participants