Skip to content
This repository has been archived by the owner on Nov 8, 2018. It is now read-only.

Multiple Display of single Post Instance #351

Open
2 tasks
valerycodes opened this issue Feb 11, 2018 · 11 comments
Open
2 tasks

Multiple Display of single Post Instance #351

valerycodes opened this issue Feb 11, 2018 · 11 comments

Comments

@valerycodes
Copy link

Description

A single post should display just a single instance. Remove multiply display of a particular Post instance in the FirstAide - safety tools app.

Mocks

multiple_post

Definition of Done

  • All of the required items are completed.
  • Approval by 1 mentor.
@medhach
Copy link

medhach commented Feb 15, 2018

@valerycodes These are not same instances. The Safety Tools page of the First Aide mobile app has 5 different categories and the posts in each level will represent each category.
You could add the headers for each section here so as to avoid the obvious confusion for new coders.
You can also improve the UI of this page to show the category names and posts when clicked on them, or if you have other ideas.
Please let me know if you'd like to work on that. We will discuss.

@Tahirih-Natang
Copy link

@valerycodes @medhach I also thought of it instances of thesame post.

IMO I think it should just be one form on the page, with a post category name as an option for the user to choose when creating a post

@medhach medhach changed the title Multiply Display of single Post Instance Multiple Display of single Post Instance Feb 16, 2018
@medhach
Copy link

medhach commented Feb 16, 2018

@valerycodes Can you prepare mock screens for the same so we can review?

@valerycodes
Copy link
Author

@medhach are asking me to prepare the mock up based on @Tahirih-Natang proposal ?

@Tahirih-Natang
Copy link

Tahirih-Natang commented Feb 18, 2018

@valerycodes @medhach , well I don't think this issue need another mockup since the feature has already been implemented. What needs to be done is to call the category name in the views instead of category id. Then clean up the views so that only one form should appear at a time.
npost

@medhach
Copy link

medhach commented Feb 19, 2018

@Tahirih-Natang I was asking for a mockup of the page with a list of all posts, what you've shown here is already implemented for the new post.
@valerycodes You can think of your own design if you have any too. The idea is to display the posts of each category separately. You could do it with one drop down to select category and display just those posts, or you could have a clickable list with the category names and it can expand to show the list of each category when clicked(I personally think this would be more user-friendly). A mock is only for us to see and decide whether its the best fit for the application.

@Tahirih-Natang
Copy link

@medhach It has been implementted but wrongly done. The current implemtation requires users to memorize the different post ids which practically impossible to do. Maybe I should open a separate issue for that.

@medhach
Copy link

medhach commented Feb 25, 2018

@Tahirih-Natang Yes, you can open another issue for that. We will fix it.
And @valerycodes Have you got an idea of what I was trying to say?

@valerycodes
Copy link
Author

Yea, @medhach Still figuring out the best way of representing a proper solution for it 😄

@medhach
Copy link

medhach commented Mar 6, 2018

@valerycodes Have you thought of any solution yet? Let me know your status on this one.

@valerycodes
Copy link
Author

@medhach I will present the mock up soon

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Development

No branches or pull requests

3 participants