Skip to content

Updated app constants, moved typedefs to src/lib/types/appConstants.j…#92

Merged
SunDevil311 merged 10 commits intomasterfrom
feature/add-typedefs
Jun 4, 2025
Merged

Updated app constants, moved typedefs to src/lib/types/appConstants.j…#92
SunDevil311 merged 10 commits intomasterfrom
feature/add-typedefs

Conversation

@SunDevil311
Copy link
Member

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Description

Updated app constants, moved typedefs to src/lib/types/appConstants.js. Updated references in Badges.svelte, LegalNav.svelte, and MetaTags.svelte. Complete refactor of PostHog to utilize reactive stores with src/lib/stores/posthog.js. Updated Privacy Policy, Privacy Dashboard, +layout.svelte, as well as other supporting files, accordingly.

Checklist

  • I have read and followed the guidelines in the CONTRIBUTING document.
  • I've checked to make sure there aren't existing Pull Requests for the same update/change.
  • My code follows the code style of this project.
  • My code has been linted locally prior to submission.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • Where applicable, I have added tests to cover my changes.
  • All new and existing tests passed.

Pull requests should be thought of as a conversation. There will be some back and forth when trying to get code merged into this or any other project.

With all but the simplest changes, you can and should expect that the maintainers of the project will request changes to your code.

Please be aware of that and check in after you open your PR in order to get your code merged in cleanly.

Thank you!

…s. Updated references in Badges.svelte, LegalNav.svelte, and MetaTags.svelte. Complete refactor of PostHog to utilize reactive stores with src/lib/stores/posthog.js. Updated Privacy Policy, Privacy Dashboard, +layout.svelte, as well as other supporting files, accordingly.
@netlify
Copy link

netlify bot commented Jun 4, 2025

Deploy Preview for networkpro-web ready!

Name Link
🔨 Latest commit 5d6cf8c
🔍 Latest deploy log https://app.netlify.com/projects/networkpro-web/deploys/684002c57c5d310008917911
😎 Deploy Preview https://deploy-preview-92--networkpro-web.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@SunDevil311 SunDevil311 self-assigned this Jun 4, 2025
Copy link
Member Author

@SunDevil311 SunDevil311 left a comment

Choose a reason for hiding this comment

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

Deploy preview is showing 500 Internal Error. Merge on hold until fixed.

Copy link
Member Author

@SunDevil311 SunDevil311 left a comment

Choose a reason for hiding this comment

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

All changes look good. Merging.

@SunDevil311 SunDevil311 merged commit 939e8a0 into master Jun 4, 2025
11 checks passed
@SunDevil311 SunDevil311 deleted the feature/add-typedefs branch June 4, 2025 08:41
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.

1 participant