[Bug] Register for event doesn't work

[yasharth291]

when clicked on register for event the screen only loads. And stucks at a place.

[yasharth291]

@Sagar2366

[pr4nshul]

Hey @yasharth291 are you working on it? If not @Sagar2366 can you assign me this. Also, adding an event is also not working at least on my end. So should I create a new issue for the same?

[Sagar2366]

@pr4nshul you can start working on this issue.
About add event there is open issue #426 @CoderMayhem working on it.

[pr4nshul]

Ok, got it

[pr4nshul]

I was thinking we should also add a button to unregister in the drop down.

[pr4nshul]

@Sagar2366 Close this issue as you mentioned it is solved in previous master changes.
But I still think , that small UI improvement will be helpful for user :)

[Sagar2366]

@Sagar2366 Close this issue as you mentioned it is solved in previous master changes.
But I still think , that small UI improvement will be helpful for user :)

@pr4nshul be clear when you say small UI improvement. If you are talking about registering for the event it is already done and don't need any UI change. But if you are referring to unregister option then please continue discussion on slack. We welcome new ideas from students and hear everyone's opinions. At the same time we expect everyone to follow the guidelines - discuss idea first in slack channels and based on discussion create new issue.

Avoid suggesting new ideas/new issue discussions here in another issue. As explained you earlier if makes difficult to keep track of ongoing issue.

[pr4nshul]

Ok so what i mean bt small UI improvement is , when the user clicks the register button the screen doesn't refresh instantly and it displays you are not registered until you refresh . I will send a video for the same in few minutes. And with the changes that I made that on the PR the page get reloaded as soon as the user registers for the event. Hence , user doesn't need to refresh manually to see whether he was able to register for the event or not.

I was not talking about the unregister button. I won't suggest new ideas under any issue anymore as you told me before. Next time I will be as descriptive as I can be , sorry for the ambiguity caused.

[pr4nshul]

20210326_113240.mp4

This is on current master

[pr4nshul]

20210325_161213.mp4

This is after the changes I have in PR #509

[pr4nshul]

Hey @Sagar2366 , did you look at this? If you think this change is redundant then I would close this issue.

[yasharth291]

Is it solved @pr4nshul @Sagar2366 sir

[pr4nshul]

The button is working as intended, we just need proper state management to update the user that they are registered . So, you can close the issue, I guess.

[yasharth291]

@Sagar2366 sir can you please close it