Skip to content
This repository has been archived by the owner on Oct 31, 2023. It is now read-only.

Update to imports and some minor changes done #82

Merged
merged 3 commits into from
Nov 13, 2021
Merged

Conversation

akshaypithadiya
Copy link
Collaborator

@akshaypithadiya akshaypithadiya commented Nov 13, 2021

This closes issue #82

@@ -6,7 +6,7 @@ import Error from './Error';
import ExploreEntityList from './explore/ExploreEntityList';
import ExplorePeopleList from './explore/ExplorePeopleList';
import ExploreEntity from './explore/ExploreEntity';
import PhotoImageList from './PhotoImageList';
import FaceList from './FaceList';
Copy link
Owner

Choose a reason for hiding this comment

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

I would name it PeopleList

@prabhuomkar
Copy link
Owner

Please remove Archive from SideNav

{data.map((src) => (
<ImageListItem
key={src.id}
style={type === 'people' ? stylePeopleList : stylePlacesThigsList}
style={type === 'people' ? stylePeople : stylePlacesThigs}
Copy link
Owner

Choose a reason for hiding this comment

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

stylePlacesThings

@akshaypithadiya akshaypithadiya merged commit e7b08cb into master Nov 13, 2021
@akshaypithadiya akshaypithadiya deleted the refactor branch November 13, 2021 14:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants