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.

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

Choose layer fields to download before saving file #506

Draft
wants to merge 28 commits into
base: dev
Choose a base branch
from

Conversation

volterra79
Copy link
Member

@volterra79 volterra79 commented Oct 16, 2023

Waiting for customer approval

Ref. ticket: #cu-863h2j7pv

Depends on: g3w-suite/g3w-admin#619

fields-2023-10-17_12.17.48.mp4

How to test

Set the layer as downloadable and follow the steps of the video above:

image

@volterra79 volterra79 added this to the v3.9 milestone Oct 16, 2023
@volterra79 volterra79 self-assigned this Oct 16, 2023
@volterra79 volterra79 marked this pull request as ready for review October 17, 2023 10:15
@volterra79 volterra79 added the feature New feature or request label Oct 17, 2023
Copy link
Collaborator

@Raruto Raruto left a comment

Choose a reason for hiding this comment

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

Personal reminder:

  • update translations before merge
  • try to extrapolate download logic into a utility function (or consolidate it into a single class, e.g. Catalog or Layer store)

@Raruto Raruto changed the title Show fields choose interface when download layer format file Choose layer fields to download before saving file Oct 18, 2023
@Raruto Raruto modified the milestones: v3.9, v3.10 Oct 19, 2023
@volterra79 volterra79 marked this pull request as draft February 1, 2024 10:06
@Raruto Raruto modified the milestones: v3.10, v3.11 Apr 15, 2024
@Raruto Raruto removed this from the v3.11 milestone Jul 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants