data-umbrella / event-board-web

Event Board: front-end code
https://events.dataumbrella.org
GNU Affero General Public License v3.0
10 stars 13 forks source link

Fixes Register Button on Event Page. #348

Closed Rohanfizz closed 1 year ago

Rohanfizz commented 1 year ago

PR Overview

@jtorreggiani Please review this as I can't test with dummy backend.

Type of contribution

Reference: Issue or Pull Request (PR)

Ticket Description

342

Description of Changes

Enclosed div tag inside anchor tag.

Before and After for UI Updates

No UI changes.

For PR Reviewer

reshamas commented 1 year ago

Hi @Rohanfizz Thanks for this PR. For "342" in the Ticket description,

  1. can you check the box "Towards xxx"?
  2. select "Code contribution"?

If the PR template is not clear, please let @SangamSwadiK and I know, we can update it.

reshamas commented 1 year ago

@Rohanfizz Where it says "Towards #xxx", can you replace it with "Towards #342"?

I wonder if we should remove the "Ticket Description" section all together.

SangamSwadiK commented 1 year ago

@Rohanfizz Where it says "Towards #xxx", can you replace it with "Towards #342"?

I wonder if we should remove the "Ticket Description" section all together.

I guess having Ticket Description, saves time while reviewing the PR? as the reviewer need not get into the parent and other child issues.

But like you mentioned, there may be a better way to do it.

reshamas commented 1 year ago

@SangamSwadiK Another option, only have "Towards #xxx". Then, we can manually close the issue. This way, it links the PR to the issue. And it is not too much for a contributor to read.

I thought our template would make it easier for contributors, but it may be complicating things. too much text there.

SangamSwadiK commented 1 year ago

@SangamSwadiK Another option, only have "Towards #xxx". Then, we can manually close the issue. This way, it links the PR to the issue. And it is not too much for a contributor to read.

I thought our template would make it easier for contributors, but it may be complicating things. too much text there.

Yes, we can do this.

jtorreggiani commented 1 year ago

@Rohanfizz, since this got stale and the tests were breaking I pushed up fixes to unblock this. Let's sync up on some of the other open tickets.