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

605: Add card extensions #610

Merged
merged 44 commits into from
Dec 12, 2022
Merged

605: Add card extensions #610

merged 44 commits into from
Dec 12, 2022

Conversation

maxammann
Copy link
Member

@maxammann maxammann commented Nov 1, 2022

Changes:

  • Switch from Long package to just bigint (graphql + protobuf)
  • Migrate away from protobufjs because of many unsolved issues....
    • Install protoc!
  • Adjust code to new protobuf spec
  • Remove usage of model variable name
  • Change from seconds to days for expiration ⚠️

@maxammann maxammann force-pushed the 605-card-extensions branch from 49b428a to 41017e1 Compare December 5, 2022 10:56
@maxammann maxammann force-pushed the 605-card-extensions branch from 41017e1 to 63f1293 Compare December 5, 2022 13:56
@maxammann maxammann force-pushed the 605-card-extensions branch from 09af7c4 to 8dd6b48 Compare December 5, 2022 14:36
@maxammann maxammann marked this pull request as ready for review December 5, 2022 14:55
administration/src/components/generation/PdfFactory.ts Outdated Show resolved Hide resolved
administration/src/components/generation/PdfFactory.ts Outdated Show resolved Hide resolved
administration/src/util/day.ts Outdated Show resolved Hide resolved
administration/src/util/generateHashFromCardDetails.ts Outdated Show resolved Hide resolved
specs/card.proto Outdated Show resolved Hide resolved
specs/card.proto Outdated Show resolved Hide resolved
specs/card.proto Show resolved Hide resolved
specs/card.proto Outdated Show resolved Hide resolved
frontend/pubs/df_protobuf/pubspec.yaml Outdated Show resolved Hide resolved
@maxammann maxammann enabled auto-merge (squash) December 12, 2022 16:20
@maxammann maxammann merged commit d49ac17 into whitelabel Dec 12, 2022
@maxammann maxammann deleted the 605-card-extensions branch December 12, 2022 16:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants