anitab-forms-web icon indicating copy to clipboard operation
anitab-forms-web copied to clipboard

Profile Form doesnt close after submitting

Open ruddi10 opened this issue 3 years ago • 14 comments

Describe the bug

When a user edit or fill the profile even after pressing the submit button the form remains open and I think that creates confusion I would rather suggest that we close the form .

To Reproduce

Steps to reproduce the behavior:

  1. Go to dashboard
  2. Click on Edit Profile
  3. Change some information and click Submit button
  4. The form will not close

Expected behavior

I think we should close the form and moreover can use a tost notification once the profile is updated.

Screenshots

Screenshot from 2021-03-08 17-43-44

Expected- Screenshot from 2021-03-08 17-44-38

ruddi10 avatar Mar 08 '21 12:03 ruddi10

If this issue is fine. Kindly assign it to me @codesankalp @keshakaneria

ruddi10 avatar Mar 08 '21 12:03 ruddi10

Sure @ruddi10. I am assigning this to you. Also, one suggestion, If after submission backends respond with an error then don't close the form and display the error inside a message box. Also, don't forget to write a test for this feature.

codesankalp avatar Mar 08 '21 12:03 codesankalp

Also should I show a pop-up incase the update is successful or not? @codesankalp

ruddi10 avatar Mar 08 '21 12:03 ruddi10

Also should I show a pop-up incase the update is successful or not? @codesankalp

Pop up or a line in red on top above Dashboard text will be fine to let the user know if the form is submitted.

keshakaneria avatar Mar 08 '21 14:03 keshakaneria

Also should I show a pop-up incase the update is successful or not? @codesankalp

Inline Error message box which disappears after some time. Use Semantic UI.

codesankalp avatar Mar 08 '21 16:03 codesankalp

I was asking incase its successful

ruddi10 avatar Mar 08 '21 18:03 ruddi10

Also should I show a pop-up incase the update is successful or not? @codesankalp

Pop up or a line in red on top above Dashboard text will be fine to let the user know if the form is submitted.

You can go for this @ruddi10. If you have any queries do let us know.

keshakaneria avatar Mar 08 '21 18:03 keshakaneria

I was asking incase its successful

Same success message box provided in Semantic UI. And as mentioned by @keshakaneria you can go with your own idea and we will review it in PR.

codesankalp avatar Mar 08 '21 18:03 codesankalp

@keshakaneria @codesankalp I have fixed this issue just wanted to know the code formatter used for this repo as I use Prettier and its changing the formatting on save so it might be a problem

ruddi10 avatar Mar 09 '21 12:03 ruddi10

This will solve after merging of https://github.com/anitab-org/anitab-forms-web/pull/94. You can make PR with formatted code.

codesankalp avatar Mar 09 '21 13:03 codesankalp

ok thanks

ruddi10 avatar Mar 09 '21 13:03 ruddi10

Can I work on this? Can you please guide me since I am new

eleensmathew avatar Feb 14 '22 12:02 eleensmathew

If the issue is still there, kindly assign it to me.

jivteshsingh avatar Aug 21 '22 20:08 jivteshsingh

If this issue is still active, I can work on it

Bishwajitpvt avatar Aug 28 '23 16:08 Bishwajitpvt