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

Internal components refactor [g3w-client #574] #787

Merged
merged 39 commits into from
Apr 22, 2024
Merged

Conversation

Raruto
Copy link
Contributor

@Raruto Raruto commented Mar 22, 2024

List of changes

  1. internal JS/Vue components refactor: Refactor G3WObject, Panel and Component classes g3w-client#574
  2. refactor search config API (ref: WidgetSerializer and /api/config/<gid>/qdjango/<pid>)
  3. deprecate setting G3W_CLIENT_SEARCH_ENDPOINT = 'ows' (until removal, the new default is: 'api')

@Raruto Raruto added the refactoring Anything which could result in a API change label Mar 22, 2024
@Raruto Raruto added this to the v3.8 milestone Mar 22, 2024
'type': ({
'autocompletebox': 'autocompletefield',
'selectbox': 'selectfield'
}).get(field.get('widgettype'), field['input'].get('type')),
Copy link
Contributor Author

@Raruto Raruto Mar 25, 2024

Choose a reason for hiding this comment

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

@wlorenzetti the numberfield type should also be returned in here (currently not implement in v3.7 response):

OK: "Volume greater than" appears as number field

image

KO: "Volume greater than" appears as text field

image

Sample config: "Type and volume"

image

@Raruto Raruto marked this pull request as ready for review April 9, 2024 13:41
@Raruto Raruto requested a review from wlorenzetti April 9, 2024 13:44
@Raruto
Copy link
Contributor Author

Raruto commented Apr 18, 2024

@leolami g3w-suite/g3w-client@f99134b takes care of empty query result for: autompletefield + RelationReference.

Before

image

After

image


Unsearchable layer(s)

I think some check is missing on the server side.

From what I see, some other layers shouldn't be searchable either (unless relationships could break that option?):

image

@Raruto Raruto requested review from wlorenzetti and removed request for wlorenzetti April 19, 2024 08:52
@Raruto
Copy link
Contributor Author

Raruto commented Apr 22, 2024

@wlorenzetti still waiting for your review → merge 😉

Copy link
Member

@wlorenzetti wlorenzetti left a comment

Choose a reason for hiding this comment

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

I seems works fine, I just make some tests.

@wlorenzetti wlorenzetti merged commit b6be288 into dev Apr 22, 2024
4 checks passed
@wlorenzetti wlorenzetti deleted the g3w-client_PR574 branch April 22, 2024 08:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactoring Anything which could result in a API change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants