Skip to content

feat: add create group page#601

Merged
karantakalkar merged 1 commit intoopenMF:masterfrom
karantakalkar:create-group
May 22, 2020
Merged

feat: add create group page#601
karantakalkar merged 1 commit intoopenMF:masterfrom
karantakalkar:create-group

Conversation

@karantakalkar
Copy link
Copy Markdown
Collaborator

Description

Added routing to 'Create Group' button in Group's List View, designed the UI for Create Group Form
and connected it to the API.

Related issues and discussion

#212

Screenshots, if any

UI

Screenshot from 2020-02-08 18-49-31

Checklist

Please make sure these boxes are checked before submitting your pull request - thanks!

  • If you have multiple commits please combine them into one commit by squashing them.

  • Read and understood the contribution guidelines at web-app/.github/CONTRIBUTING.md.

@karantakalkar
Copy link
Copy Markdown
Collaborator Author

karantakalkar commented Feb 8, 2020

Data Tables are not functional in demo.mifos.io beacuse the API response (post create group button-press) has no field for data-tables,
Could not find reference to code out data-tables feature.

Comment thread src/app/groups/create-group/create-group.component.ts Outdated
karantakalkar pushed a commit to karantakalkar/web-app that referenced this pull request Feb 22, 2020
Comment thread src/app/groups/create-group/create-group.component.html
karantakalkar pushed a commit to karantakalkar/web-app that referenced this pull request Feb 26, 2020
Comment thread src/app/groups/create-group/create-group.component.ts Outdated
Comment thread src/app/groups/groups.service.ts Outdated
Comment thread src/app/groups/create-group/create-group.component.ts Outdated
spider0061 added a commit to spider0061/web-app that referenced this pull request Mar 1, 2020
added relevant comments ,removed the unnecessary white spaces and added white spaces whereever requried.

Fixes: openMF#601
karantakalkar pushed a commit to karantakalkar/web-app that referenced this pull request Mar 7, 2020
Copy link
Copy Markdown
Collaborator

@muskankhedia muskankhedia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The UI looks nice. But when I tried to submit the form, it's displaying an error. Can you have a look if its the problem with the API or the data format that has been sent. Please attach a working GIF with the description.

Comment thread src/app/groups/groups.module.ts Outdated
Comment thread src/app/groups/create-group/create-group.component.ts
Comment thread src/app/groups/create-group/create-group.component.ts Outdated
Comment thread src/app/groups/create-group/create-group.component.html
Comment thread src/app/groups/create-group/create-group.component.html
@karantakalkar karantakalkar added the review requested Pull Request requiring review by senior collaborators label May 12, 2020
@karantakalkar
Copy link
Copy Markdown
Collaborator Author

@muskankhedia there are data tables to be implemented in this component as per communtiy app code, however they do not work in demo.mifos.io/staging.mifos.io, It functions similar to these servers though.

@muskankhedia
Copy link
Copy Markdown
Collaborator

Okay, that's fine. You can raise a JIRA ticket for the same if there is an error from the backend. Till then you can resolve the front-end issues.

karantakalkar pushed a commit to karantakalkar/web-app that referenced this pull request May 22, 2020
@karantakalkar karantakalkar removed the review requested Pull Request requiring review by senior collaborators label May 22, 2020
@karantakalkar karantakalkar merged commit 68c30cf into openMF:master May 22, 2020
@karantakalkar karantakalkar deleted the create-group branch June 23, 2020 11:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants