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

feat: build ink web app in ci #17

Merged
merged 2 commits into from
Feb 6, 2025
Merged

Conversation

ink-victor
Copy link
Contributor

@ink-victor ink-victor commented Jan 31, 2025

Changes:

  • Set up Dockerfile for ink-web-app
  • Set up CI to build, push and deploy updates on pushes to main, ops/preview-1 and ops/preview-2
  • Set up schema for our environment variables to make them predictable
  • Improved .env.example with a better structure
  • Created scripts/generate-dummy-dotenv.sh script to populate .env.production for CI jobs like lint and format that do not need the real secrets
  • Updated README.md with better content
  • Replaced ink-landingpage Sentry project with new ink-web-app project
  • Fixed sanitize/dompurify errors

Notes:

  • app-ink-web-app is going to be updated on feat/ink-web-app branch. The main reason being that main is protected and required signed commits.
image

@ink-victor ink-victor requested a review from a team as a code owner January 31, 2025 16:35
@ink-victor ink-victor force-pushed the feat/build-ink-web-app-in-ci branch 7 times, most recently from 53a5185 to a875b63 Compare February 3, 2025 16:22
@ink-victor ink-victor force-pushed the feat/build-ink-web-app-in-ci branch from c5142dc to cfd6a1c Compare February 3, 2025 17:06
@ink-victor ink-victor force-pushed the feat/build-ink-web-app-in-ci branch from cfd6a1c to 1a89f3f Compare February 3, 2025 17:19
@ink-victor ink-victor force-pushed the feat/build-ink-web-app-in-ci branch from 1a89f3f to 37bfd08 Compare February 3, 2025 17:56
@ink-victor ink-victor force-pushed the feat/build-ink-web-app-in-ci branch from 37bfd08 to e4df11b Compare February 3, 2025 18:06
@ink-victor ink-victor force-pushed the feat/build-ink-web-app-in-ci branch from e4df11b to 2158b1c Compare February 3, 2025 18:23
@ink-victor ink-victor force-pushed the feat/build-ink-web-app-in-ci branch from 2158b1c to c6d9670 Compare February 3, 2025 20:45
@ink-victor ink-victor force-pushed the feat/build-ink-web-app-in-ci branch from 4e5692a to aa6ee42 Compare February 3, 2025 21:39
@ink-victor ink-victor force-pushed the feat/build-ink-web-app-in-ci branch from aa6ee42 to e7cdfba Compare February 4, 2025 16:01
@ink-victor ink-victor force-pushed the feat/build-ink-web-app-in-ci branch 2 times, most recently from 6ea8b11 to a1d303e Compare February 5, 2025 11:14
@ink-victor ink-victor force-pushed the feat/build-ink-web-app-in-ci branch from a1d303e to b18b133 Compare February 5, 2025 11:58
@ink-victor ink-victor changed the title draft: feat: build ink web app in ci feat: build ink web app in ci Feb 5, 2025
@ink-victor ink-victor force-pushed the feat/build-ink-web-app-in-ci branch 2 times, most recently from 64e44ba to c71789c Compare February 5, 2025 20:39
@ink-victor ink-victor force-pushed the feat/build-ink-web-app-in-ci branch from c71789c to 0b0b5c9 Compare February 5, 2025 20:41
@ink-victor ink-victor force-pushed the feat/build-ink-web-app-in-ci branch from da4839d to 04592bf Compare February 5, 2025 22:10
@ink-victor ink-victor force-pushed the feat/build-ink-web-app-in-ci branch from 04592bf to 2e1f48e Compare February 5, 2025 22:13
@ink-victor ink-victor merged commit 2b68b21 into main Feb 6, 2025
4 checks passed
@ink-victor ink-victor deleted the feat/build-ink-web-app-in-ci branch February 6, 2025 12:40
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.

2 participants