-
Notifications
You must be signed in to change notification settings - Fork 3
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
build: Migrate views to Phoenix 1.7 #343
base: master
Are you sure you want to change the base?
Conversation
fd76a63
to
56bf6e2
Compare
@ruioliveira02 I agree. |
8d5fe77
to
0133865
Compare
0133865
to
307b713
Compare
token_balance: at.token_balance, | ||
prizes: for(p <- at.prizes, do: PrizeJSON.prize_attendee(%{prize: p})), | ||
entries: at.entries, | ||
redeemables: for(r <- at.redeemables, do: RedeemableJSON.my_readeemables(%{redeemable: r})) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
redeemables: for(r <- at.redeemables, do: RedeemableJSON.my_readeemables(%{redeemable: r})) | |
redeemables: for(r <- at.redeemables, do: RedeemableJSON.my_redeemables(%{redeemable: r})) |
Fix the typo in this line
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
def render("price.json", %{price: price}) do | ||
%{price: price} | ||
end |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think is due to this function you should create
def price(%{price: price}) do
%{price: price}
end
4e8ee4f
to
16499f3
Compare
Closes #339.
Admin controllers aren't migrated. I think we should remove them and start from scratch there. What do you think @RuiL1904 @MarioRodrigues10 ?