Skip to content

[Dumfries] Add itemsTitle from asset layer to Enquiry#514

Open
MorayMySoc wants to merge 1 commit into
masterfrom
5473-dumfries-set-location
Open

[Dumfries] Add itemsTitle from asset layer to Enquiry#514
MorayMySoc wants to merge 1 commit into
masterfrom
5473-dumfries-set-location

Conversation

@MorayMySoc

Copy link
Copy Markdown
Contributor

Set the location field of the Alloy Enquiry to contain the title from the asset layer

https://github.com/mysociety/societyworks/issues/5473

@MorayMySoc MorayMySoc requested a review from dracos May 15, 2026 11:55
Comment thread perllib/Open311/Endpoint/Integration/UK/Dumfries.pm Outdated
Comment thread perllib/Open311/Endpoint/Integration/UK/Dumfries.pm Outdated
Comment thread perllib/Open311/Endpoint/Integration/UK/Dumfries.pm Outdated
@MorayMySoc MorayMySoc requested a review from dracos May 15, 2026 13:32
my $attributes = $self->SUPER::process_attributes($args);
#If there is an asset_resource_id will have cached the call
# for the item in set_parent_attribute
my $asset_attributes = $args->{attributes}->{asset_resource_id}

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

If I've assigned this, I should probably be using $asset_attributes going forward...

#If there is an asset_resource_id will have cached the call
# for the item in set_parent_attribute
my $asset_attributes = $args->{attributes}->{asset_resource_id}
? $self->api_cache->{ $args->{attributes}->{asset_resource_id} }

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Ah, ok. Doesn't work here...

@mysociety-pusher mysociety-pusher force-pushed the 5473-dumfries-set-location branch from dc76498 to 1c17751 Compare May 19, 2026 14:06

@dracos dracos left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks

Set the location field of the Alloy Enquiry to contain
the title from the asset layer

mysociety/societyworks#5473
@mysociety-pusher mysociety-pusher force-pushed the 5473-dumfries-set-location branch from 1c17751 to a8a1a62 Compare May 21, 2026 09:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants