Skip to content
This repository has been archived by the owner on Oct 11, 2024. It is now read-only.

TT-99 Policies update #30

Merged
merged 2 commits into from
Jun 10, 2020
Merged

TT-99 Policies update #30

merged 2 commits into from
Jun 10, 2020

Conversation

tbuchaillot
Copy link

@tbuchaillot tbuchaillot commented May 11, 2020 •
edited
Loading

Description

Update of policy struct to avoid losing data between dump and sync.

Solves #28.

Testing

  1. Create a policy with metadata
  2. dump that policy.
  3. Delete the policy.
  4. sync the policy again
  5. Check if the metadata persists.

@l3ender
Copy link

l3ender commented May 28, 2020

Are these able to be merged and released? Thanks!

@buger
Copy link
Member

buger commented May 28, 2020

Yes we are planning Sync release next week.

@tbuchaillot tbuchaillot mentioned this pull request Jun 1, 2020
Closed
@tbuchaillot tbuchaillot linked an issue Jun 1, 2020 that may be closed by this pull request
tyk-sync dump does not export policy metadata #28
Closed
@tbuchaillot tbuchaillot self-assigned this Jun 5, 2020
@matiasinsaurralde matiasinsaurralde merged commit 68fc202 into master Jun 10, 2020
@matiasinsaurralde matiasinsaurralde deleted the policies-update branch June 10, 2020 13:28
@gregdelhon gregdelhon changed the title Policies update TT-99 Policies update Sep 3, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@matiasinsaurralde matiasinsaurralde matiasinsaurralde approved these changes

Assignees

@tbuchaillot tbuchaillot

Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

tyk-sync dump does not export policy metadata
4 participants
@tbuchaillot @l3ender @buger @matiasinsaurralde