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

CSV Download #3061

Merged
merged 2 commits into from
Nov 20, 2024
Merged

CSV Download #3061

merged 2 commits into from
Nov 20, 2024

Conversation

allenpigar
Copy link
Contributor

Should be deploy the gcp-cf first before deploying this pr : https://github.com/zesty-io/gcp-cf/pull/179

added export csv function on content item.
csv-download

@allenpigar allenpigar requested review from shrunyan, agalin920 and finnar-bin and removed request for shrunyan and agalin920 November 18, 2024 23:39
@allenpigar allenpigar added the feature Additional functionality that should be added to Zesty label Nov 18, 2024
@shrunyan shrunyan merged commit cb1fd29 into dev Nov 20, 2024
1 check failed
@shrunyan shrunyan deleted the feat/csv-download branch November 20, 2024 23:05
}
>({
query: ({ modelZUID }) => {
return {
Copy link
Contributor

Choose a reason for hiding this comment

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

@allenpigar the users session is not being sent with this request causing it to fail.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

user session is added using the preparedHeaders in utils: https://github.com/zesty-io/manager-ui/pull/3061/files#diff-205e3a012094d2e6f027f951e9e9e609afaccb2cc2bbe29850f56c52d8e4db9dR2

i already made adjustment on the env variable in dev. seems like the env was not set properly on lower env which causes the invalid session error.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Additional functionality that should be added to Zesty
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants