Remove address_type and rename phone to phone_address

An event can indeed have several address types

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
This commit is contained in:
Thomas Citharel 2019-01-14 17:48:08 +01:00
parent b0bc8dfa5d
commit 289ba03960
No known key found for this signature in database
GPG key ID: A061B9DDE0CA0773
17 changed files with 93 additions and 24 deletions

View file

@ -105,9 +105,9 @@
</v-list-tile-action> </v-list-tile-action>
<v-list-tile-content> <v-list-tile-content>
<v-list-tile-title><span v-if="event.address_type === 'physical'"> <v-list-tile-title>
{{ event.physical_address.streetAddress }} {{ event.physical_address.streetAddress }}
</span></v-list-tile-title> </v-list-tile-title>
<v-list-tile-sub-title>Mobile</v-list-tile-sub-title> <v-list-tile-sub-title>Mobile</v-list-tile-sub-title>
</v-list-tile-content> </v-list-tile-content>
</v-list-tile> </v-list-tile>

View file

@ -12,13 +12,12 @@ export const FETCH_EVENT = gql`
ends_on, ends_on,
state, state,
status, status,
public, visibility,
thumbnail, thumbnail,
large_image, large_image,
publish_at, publish_at,
# address_type,
# online_address, # online_address,
# phone, # phone_address,
organizerActor { organizerActor {
avatarUrl, avatarUrl,
preferredUsername, preferredUsername,
@ -56,13 +55,12 @@ export const FETCH_EVENTS = gql`
ends_on, ends_on,
state, state,
status, status,
public, visibility,
thumbnail, thumbnail,
large_image, large_image,
publish_at, publish_at,
# address_type,
# online_address, # online_address,
# phone, # phone_address,
organizerActor { organizerActor {
avatarUrl, avatarUrl,
preferredUsername, preferredUsername,

View file

@ -1,5 +1,4 @@
import EctoEnum import EctoEnum
defenum(Mobilizon.Events.AddressTypeEnum, :address_type, [:physical, :url, :phone, :other])
defenum(Mobilizon.Events.EventVisibilityEnum, :event_visibility_type, [ defenum(Mobilizon.Events.EventVisibilityEnum, :event_visibility_type, [
:public, :public,
@ -38,9 +37,8 @@ defmodule Mobilizon.Events.Event do
field(:large_image, :string) field(:large_image, :string)
field(:publish_at, Timex.Ecto.DateTimeWithTimezone) field(:publish_at, Timex.Ecto.DateTimeWithTimezone)
field(:uuid, Ecto.UUID, default: Ecto.UUID.generate()) field(:uuid, Ecto.UUID, default: Ecto.UUID.generate())
field(:address_type, Mobilizon.Events.AddressTypeEnum, default: :physical)
field(:online_address, :string) field(:online_address, :string)
field(:phone, :string) field(:phone_address, :string)
belongs_to(:organizer_actor, Actor, foreign_key: :organizer_actor_id) belongs_to(:organizer_actor, Actor, foreign_key: :organizer_actor_id)
belongs_to(:attributed_to, Actor, foreign_key: :attributed_to_id) belongs_to(:attributed_to, Actor, foreign_key: :attributed_to_id)
many_to_many(:tags, Tag, join_through: "events_tags") many_to_many(:tags, Tag, join_through: "events_tags")
@ -69,9 +67,8 @@ defmodule Mobilizon.Events.Event do
:thumbnail, :thumbnail,
:large_image, :large_image,
:publish_at, :publish_at,
:address_type,
:online_address, :online_address,
:phone :phone_address
]) ])
|> cast_assoc(:tags) |> cast_assoc(:tags)
|> cast_assoc(:physical_address) |> cast_assoc(:physical_address)
@ -82,8 +79,7 @@ defmodule Mobilizon.Events.Event do
:organizer_actor_id, :organizer_actor_id,
:category_id, :category_id,
:url, :url,
:uuid, :uuid
:address_type
]) ])
end end

View file

@ -4,7 +4,6 @@ defmodule MobilizonWeb.Schema do
""" """
use Absinthe.Schema use Absinthe.Schema
import Absinthe.Resolution.Helpers, only: [dataloader: 1]
alias Mobilizon.{Actors, Events} alias Mobilizon.{Actors, Events}
alias Mobilizon.Actors.{Actor, Follower, Member} alias Mobilizon.Actors.{Actor, Follower, Member}
alias Mobilizon.Events.{Event, Comment, Participant} alias Mobilizon.Events.{Event, Comment, Participant}
@ -200,9 +199,8 @@ defmodule MobilizonWeb.Schema do
arg(:thumbnail, :string) arg(:thumbnail, :string)
arg(:large_image, :string) arg(:large_image, :string)
arg(:publish_at, :datetime) arg(:publish_at, :datetime)
arg(:address_type, non_null(:address_type))
arg(:online_address, :string) arg(:online_address, :string)
arg(:phone, :string) arg(:phone_address, :string)
arg(:organizer_actor_username, non_null(:string)) arg(:organizer_actor_username, non_null(:string))
arg(:category, non_null(:string)) arg(:category, non_null(:string))

View file

@ -1,4 +1,7 @@
defmodule MobilizonWeb.Schema.ActorInterface do defmodule MobilizonWeb.Schema.ActorInterface do
@moduledoc """
Schema representation for Actor
"""
use Absinthe.Schema.Notation use Absinthe.Schema.Notation
import Absinthe.Resolution.Helpers, only: [dataloader: 1] import Absinthe.Resolution.Helpers, only: [dataloader: 1]
alias Mobilizon.Actors.Actor alias Mobilizon.Actors.Actor

View file

@ -1,4 +1,7 @@
defmodule MobilizonWeb.Schema.Actors.FollowerType do defmodule MobilizonWeb.Schema.Actors.FollowerType do
@moduledoc """
Schema representation for Follower
"""
use Absinthe.Schema.Notation use Absinthe.Schema.Notation
@desc """ @desc """

View file

@ -1,4 +1,7 @@
defmodule MobilizonWeb.Schema.Actors.GroupType do defmodule MobilizonWeb.Schema.Actors.GroupType do
@moduledoc """
Schema representation for Group
"""
use Absinthe.Schema.Notation use Absinthe.Schema.Notation
import Absinthe.Resolution.Helpers, only: [dataloader: 1] import Absinthe.Resolution.Helpers, only: [dataloader: 1]
import_types(MobilizonWeb.Schema.Actors.MemberType) import_types(MobilizonWeb.Schema.Actors.MemberType)

View file

@ -1,4 +1,7 @@
defmodule MobilizonWeb.Schema.Actors.MemberType do defmodule MobilizonWeb.Schema.Actors.MemberType do
@moduledoc """
Schema representation for Member
"""
use Absinthe.Schema.Notation use Absinthe.Schema.Notation
@desc """ @desc """

View file

@ -1,4 +1,7 @@
defmodule MobilizonWeb.Schema.Actors.PersonType do defmodule MobilizonWeb.Schema.Actors.PersonType do
@moduledoc """
Schema representation for Person
"""
use Absinthe.Schema.Notation use Absinthe.Schema.Notation
import Absinthe.Resolution.Helpers, only: [dataloader: 1] import Absinthe.Resolution.Helpers, only: [dataloader: 1]
import_types(MobilizonWeb.Schema.UserType) import_types(MobilizonWeb.Schema.UserType)

View file

@ -1,4 +1,7 @@
defmodule MobilizonWeb.Schema.AddressType do defmodule MobilizonWeb.Schema.AddressType do
@moduledoc """
Schema representation for Address
"""
use Absinthe.Schema.Notation use Absinthe.Schema.Notation
object :physical_address do object :physical_address do

View file

@ -1,4 +1,7 @@
defmodule MobilizonWeb.Schema.CommentType do defmodule MobilizonWeb.Schema.CommentType do
@moduledoc """
Schema representation for Comment
"""
use Absinthe.Schema.Notation use Absinthe.Schema.Notation
@desc "A comment" @desc "A comment"
@ -6,9 +9,23 @@ defmodule MobilizonWeb.Schema.CommentType do
field(:uuid, :uuid) field(:uuid, :uuid)
field(:url, :string) field(:url, :string)
field(:local, :boolean) field(:local, :boolean)
field(:visibility, :comment_visibility)
field(:text, :string) field(:text, :string)
field(:primaryLanguage, :string) field(:primaryLanguage, :string)
field(:replies, list_of(:comment)) field(:replies, list_of(:comment))
field(:threadLanguages, non_null(list_of(:string))) field(:threadLanguages, non_null(list_of(:string)))
end end
@desc "The list of visibility options for a comment"
enum :comment_visibility do
value(:public, description: "Publically listed and federated. Can be shared.")
value(:unlisted, description: "Visible only to people with the link - or invited")
value(:private,
description: "Visible only to people members of the group or followers of the person"
)
value(:moderated, description: "Visible only after a moderator accepted")
value(:invite, description: "visible only to people invited")
end
end end

View file

@ -1,4 +1,7 @@
defmodule MobilizonWeb.Schema.EventType do defmodule MobilizonWeb.Schema.EventType do
@moduledoc """
Schema representation for Event
"""
use Absinthe.Schema.Notation use Absinthe.Schema.Notation
import Absinthe.Resolution.Helpers, only: [dataloader: 1] import Absinthe.Resolution.Helpers, only: [dataloader: 1]
import_types(MobilizonWeb.Schema.AddressType) import_types(MobilizonWeb.Schema.AddressType)
@ -14,9 +17,8 @@ defmodule MobilizonWeb.Schema.EventType do
field(:description, :string, description: "The event's description") field(:description, :string, description: "The event's description")
field(:begins_on, :datetime, description: "Datetime for when the event begins") field(:begins_on, :datetime, description: "Datetime for when the event begins")
field(:ends_on, :datetime, description: "Datetime for when the event ends") field(:ends_on, :datetime, description: "Datetime for when the event ends")
field(:state, :integer, description: "State of the event") field(:status, :event_status, description: "Status of the event")
field(:status, :integer, description: "Status of the event") field(:visibility, :event_visibility, description: "The event's visibility")
field(:public, :boolean, description: "Whether the event is public or not")
# TODO replace me with picture object # TODO replace me with picture object
field(:thumbnail, :string, description: "A thumbnail picture for the event") field(:thumbnail, :string, description: "A thumbnail picture for the event")
# TODO replace me with banner # TODO replace me with banner
@ -36,7 +38,7 @@ defmodule MobilizonWeb.Schema.EventType do
field(:category, :category, description: "The event's category") field(:category, :category, description: "The event's category")
field(:participants, list_of(:participant), field(:participants, list_of(:participant),
resolve: &Resolvers.Event.list_participants_for_event/3, resolve: &MobilizonWeb.Resolvers.Event.list_participants_for_event/3,
description: "The event's participants" description: "The event's participants"
) )
@ -46,4 +48,24 @@ defmodule MobilizonWeb.Schema.EventType do
field(:updated_at, :datetime, description: "When the event was last updated") field(:updated_at, :datetime, description: "When the event was last updated")
field(:created_at, :datetime, description: "When the event was created") field(:created_at, :datetime, description: "When the event was created")
end end
@desc "The list of visibility options for an event"
enum :event_visibility do
value(:public, description: "Publically listed and federated. Can be shared.")
value(:unlisted, description: "Visible only to people with the link - or invited")
value(:private,
description: "Visible only to people members of the group or followers of the person"
)
value(:moderated, description: "Visible only after a moderator accepted")
value(:invite, description: "visible only to people invited")
end
@desc "The list of possible options for the event's status"
enum :event_status do
value(:tentative, description: "The event is tentative")
value(:confirmed, description: "The event is confirmed")
value(:cancelled, description: "The event is cancelled")
end
end end

View file

@ -1,4 +1,7 @@
defmodule MobilizonWeb.Schema.Events.CategoryType do defmodule MobilizonWeb.Schema.Events.CategoryType do
@moduledoc """
Schema representation for Category
"""
use Absinthe.Schema.Notation use Absinthe.Schema.Notation
@desc "A category" @desc "A category"

View file

@ -1,5 +1,9 @@
defmodule MobilizonWeb.Schema.Events.ParticipantType do defmodule MobilizonWeb.Schema.Events.ParticipantType do
@moduledoc """
Schema representation for Participant
"""
use Absinthe.Schema.Notation use Absinthe.Schema.Notation
import Absinthe.Resolution.Helpers, only: [dataloader: 1]
@desc "Represents a participant to an event" @desc "Represents a participant to an event"
object :participant do object :participant do

View file

@ -1,4 +1,7 @@
defmodule MobilizonWeb.Schema.UserType do defmodule MobilizonWeb.Schema.UserType do
@moduledoc """
Schema representation for User
"""
use Absinthe.Schema.Notation use Absinthe.Schema.Notation
@desc "A local user of Mobilizon" @desc "A local user of Mobilizon"

View file

@ -0,0 +1,11 @@
defmodule Mobilizon.Repo.Migrations.RemoveAddressType do
use Ecto.Migration
def up do
alter table(:events) do
remove(:address_type)
end
execute "DROP TYPE address_type"
rename table(:events), :phone, to: :phone_address
end
end

View file

@ -118,8 +118,7 @@ defmodule MobilizonWeb.Resolvers.EventResolverTest do
description: "it will be fine", description: "it will be fine",
begins_on: "#{DateTime.utc_now() |> DateTime.to_iso8601()}", begins_on: "#{DateTime.utc_now() |> DateTime.to_iso8601()}",
organizer_actor_username: "#{actor.preferred_username}", organizer_actor_username: "#{actor.preferred_username}",
category: "#{category.title}", category: "#{category.title}"
address_type: #{"OTHER"}
) { ) {
title, title,
uuid uuid