Add admin comments to submissions

[damianmarti]

Allow admin users to add comments to each submission.

closes #15

[damianmarti]

@technophile-04 @rin-st for review

[rin-st]

Thanks @damianmarti ! Overall working good, added some comments

And also date and textarea looking strange, I think it's better to fix it, just to not forget about them

[damianmarti]

Thanks @damianmarti ! Overall working good, added some comments

And also date and textarea looking strange, I think it's better to fix it, just to not forget about them

Yeah, sure, I didn't spend too much time with styles now and I focused on features

[damianmarti]

Thanks @damianmarti ! Overall working good, added some comments

And also date and textarea looking strange, I think it's better to fix it, just to not forget about them

@rin-st For the date and time text, do you think that makes sense to use dayjs or something like that, or can we just format the date using the regular Date functions?

[rin-st]

@rin-st For the date and time text, do you think that makes sense to use dayjs or something like that, or can we just format the date using the regular Date functions?

I think regular is enough, something like this

[damianmarti]

@rin-st For the date and time text, do you think that makes sense to use dayjs or something like that, or can we just format the date using the regular Date functions?

I think regular is enough, something like this

Yeah, but I should add hours and minutes here, I think it's important for the comments.

[rin-st]

Lgtm!

[technophile-04]

Thanks @damianmarti!! Just pushed a couple of commits.

1. Keeping the Submissions component as server component. I think you made it client component because you wanted to refetch data after mutation? We could also achieve it by using refresh from useRouter. NextJs won't refresh the whole page instead it will update the data smartly.

2. Using default network as sepolia this way we don't need to run yarn chain until we have the contract

[damianmarti]

Thanks @damianmarti!! Just pushed a couple of commits.

1. Keeping the Submissions component as server component. I think you made it client component because you wanted to refetch data after mutation? We could also achieve it by using refresh from useRouter. NextJs won't refresh the whole page instead it will update the data smartly.
2. Using default network as sepolia this way we don't need to run yarn chain until we have the contract

Thanks @technophile-04 !!

Yes, I thought it would be better to refetch only the submissions after mutation.