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

Filter vs Export versie 4.1 #1

Closed
gemguardian opened this issue Apr 15, 2024 · 8 comments
Closed

Filter vs Export versie 4.1 #1

gemguardian opened this issue Apr 15, 2024 · 8 comments
Assignees

Comments

@gemguardian
Copy link
Member

@luukverhoeven Ik loop nu tegen een probleem aan in de 4.1 versie van het blok COonfigurable report.
Bijgaand heb je een database query met diverse filters.

Wanneer we de filter gebruiken wordt dit goed getoond op de pagina zelf. Maar bij de export worden ALLE records meegenomen.
image

@gemguardian
Copy link
Member Author

Comment from Freshservice:
Freshservice: 21460 linked successfully.
Freshservice: #21460
By: Gemma Lesterhuis <[email protected]>

@luukverhoeven
Copy link
Collaborator

@gemguardian Bij de oude versie van de plug-in kwam alleen de geselecteerde data terug die binnen de filters aanwezig was?

@gemguardian
Copy link
Member Author

@luukverhoeven ik denk dat we elkaar goed begrijpen en het antwoord ja is.

In de oude versie van de Plugin maakte ik een filtering, dit werd dan op het scherm getoond zoals in de screenshot. En dan zou ik bij de Excel/CSV import alleen deze regel terug zien en niet alle regels uit de database query.

@luukverhoeven
Copy link
Collaborator

Zie nu ook je mail, daarin staat het ook dat het eerder correct werkt. Ik ga het onderzoeken, staat op de lijst.

@gemguardian
Copy link
Member Author

@luukverhoeven kun jij aangeven wanneer je hier mee aan de slag gaat?

@servicedeskltnc

@luukverhoeven
Copy link
Collaborator

@gemguardian Deze stond op de todo, maar had geen duedata :(

Het issue heb ik gevonden, kun je testen of naar wens werkt?

@gemguardian
Copy link
Member Author

@luukverhoeven dit werkt - net nu hij doorgezet heeft naar 4.1 :) Kun je deze fix pushen naar de orginele en meldde wat het issue was?

@luukverhoeven
Copy link
Collaborator

@gemguardian Pull request aangemaakt, jleyva#248

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants