Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Hotelentity #144

Open
wants to merge 110 commits into
base: v2
Choose a base branch
from
Open

Hotelentity #144

wants to merge 110 commits into from

Conversation

pshah28
Copy link

@pshah28 pshah28 commented Sep 9, 2019

No description provided.

cdworak and others added 30 commits October 26, 2018 09:34
@cdworak
List entities works
0213698
@cdworak
List entities works with custom entity types
b5059b2
@cdworak
cleaning up
33e382e
@cdworak
got json unmarshal working
a9478dd
@cdworak
EntityService: done except for techops bugs
312e769
@cdworak
Getting registry to work without Copy()
64cfaba
@cdworak
pull out registry
2ed61e8
@cdworak
Cleaning up
d1ad58e
@cdworak
PR fixes
f28a871
@cdworak
reflection fun
e5d4ccc
@cdworak
wip
4254350
@cdworak
wip
ecd438a
@cdworak
diff is ready for review
2d1aab8
@cdworak
clean up
44901a1
@cdworak
clean up 2
b0ca1b8
@cdworak
revert acl_diff and role_diff
bb7e2ef
@upalsaha
CFT Assets: modify asset data model, add asset type registry to asset…
27e7d23 
… service (#63)

* CFT Assets: modify asset data model, add asset type registry to asset service

* CFT Assets: fix value nesting for simple types

* CFT Assets: Field corrections, type assert text asset values as string, cleanup

* Entities: Registered LocationEntity by default, added new hours handling, other cleanup

* Entities: Move Location specific metadata back into LocationEntity

* Assets: Add GetId() function

* Assets: Restore old asset service endpoints

* CFT Asset Service: Add to client

* Update entity fields, update Go versions in TravisCI config
CFT Assets: Fix ImageValue struct tags
014851d
@cdworak
fix travis build
7796080
CFT Assets: Add Asset Diff() function
7630089
CFT Assets: Asset Diff() fixes
5ef9785
LocationEntity Diff: Add uint64 Zero value handling for Image height/…
4829e40 
…width fields
@cdworak
Entity Profile Service: Get
29b7de7
Location Entity: Change Closed field to bool to match API response
49d86e0
@cdworak
entity-profile-service
8dda098
@cdworak
entity-profile-service
3211cde
Update location.go to include ISORegionCode (#66)
7d46c71 
R=cdworak,usaha
Types: Add Getters for easy type dereferencing
2299d77
Types: Add Getter handling to return default primitive values for nil…
fbbcc2f 
… fields
CTF Assets: Remove ImageValue height/width fields as they aren't sett…
799c895 
…able through API
cdworak and others added 30 commits March 28, 2019 16:32
@cdworak
entity: update json tag of specialties to specialities to match Yext …
4dd13fa 
…API bug (#115)
@tbirdsall
Lat long (#116)
6094a59 
* location: added yextDisplayLat and yextDisplayLng

* location: update spacing
@pshah28
Degrees Fix (#117)
15195c9
@cdworak
customfield_service: handle null option value (#118)
d9b8b68
@xzou
Change type of logos from **Image to **Photo
2956ec3
@bhainesva
Add support for fields param (#119)
133538f
@cdworak
entity_diff: handle double pointer to nil (aka null)
77ed347
@creotutar
Add Get NPI for Healthcare Professional Entity
5b1c20a
@alexisgrow
unordered strings equal: Update null checking for interface (#122)
7e96640
@cdworak
Hours: allow for Closed or closed (#124)
569a37b
@pshah28
Add geomodifier to location entity (#127)
547bf13
@cdworak
SetLabelsWithUnorderedStrings: handle nil J=PC-43989
c72be6b
@cdworak
Register healthcare professional entity and add Get functions
e9a8a52
@cdworak
get and set custom field values by field name
23afbb1
@cdworak
updating gender to **string
864fce5
@byronharvey24
Comment labels
af2d590
@xzou
Add and register ATM entity
27e6419
@byronharvey24
Merge branch 'entities' of github.com:yext/yext-go into commentLabels
7243539
@xzou
Remove references to upper-snake-case entity types
9c8a2c5 
Done following Product update that has made entity types
returned from the CF endpoint camel case for consistency
with the entities API
@byronharvey24
Merge branch 'entities' of github.com:yext/yext-go into commentLabels
628e0ad
@pshah28
add healthcare facility entity (#136)
266822e 
* add healthcare facility entity

* remove todo
@byronharvey24
Merge branch 'commentLabels' into entities
6517472
@the-nemz
Added custom field type tests to entity differ (#140)
4b51b45 
* Added custom field type tests to entity differ

* Fix bug with null types
@the-nemz
Add getters for VideoUrl (#141)
aa98311
@cdworak
Diff() for RawEntity
24f0309
@xzou
Add constant for Yext hours format
59974c0
@xzou @pshah28
Add and register Hotel entity (#138)
9df5ae7
@pshah28
add google fields to hotel entity
538b95f
@pshah28
modify v2 loc converter to accept hotel
456e8a1
@pshah28
fix tabbing
9f4e74b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
10 participants
@pshah28 @cdworak @upalsaha @bhainesva @byronharvey24 @tbirdsall @xzou @alexisgrow @creotutar @the-nemz