Skip to content
This repository was archived by the owner on Aug 8, 2019. It is now read-only.

Commit 50d3542

Browse files
committed
Fixes with code-reviews
1 parent 82c1195 commit 50d3542

2 files changed

Lines changed: 7 additions & 8 deletions

File tree

api/app/controllers/clubs_controller.rb

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -18,8 +18,4 @@ def set_club
1818
@club = Club.find(params[:id])
1919
end
2020

21-
def club_params
22-
params.permit(:name, :email, :city, :country, :address, :image, gallery: [])
23-
end
24-
2521
end

client/src/views/owner-home.js

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ function OwnerHome() {
1515
const sportFields = useSportFields();
1616
const setClubs = useSetClubs();
1717
const setSportFields = useSetSportFields();
18-
const [activeClub, setActiveClub] = React.useState(1);
18+
const [activeClub, setActiveClub] = React.useState(null);
1919

2020
const styleClubsContainer = {
2121
display: "flex",
@@ -41,13 +41,16 @@ function OwnerHome() {
4141
React.useEffect(() => {
4242
getClubs().then(clubs => {
4343
setClubs(clubs);
44+
setActiveClub(clubs.length ? clubs[0].id : null);
4445
});
4546
}, [setClubs]);
4647

4748
React.useEffect(() => {
48-
getSportFields().then(sportFields => {
49-
setSportFields(sportFields);
50-
});
49+
if (activeClub) {
50+
getSportFields().then(sportFields => {
51+
setSportFields(sportFields);
52+
});
53+
}
5154
}, [setSportFields]);
5255

5356
const styleSportFieldsContainer = {

0 commit comments

Comments
 (0)