fix: resolve issues #116, #117, #118, #121 (Stellar Wave) - #226
Merged
LaGodxy merged 1 commit intoApr 28, 2026
Conversation
…ettaChain#121 - MettaChain#121: Add smart contract integration docs (docs/smart-contract-integration.md) covering contract addresses, ABI management, event listeners, transaction flow, error codes, and security considerations. - MettaChain#118: Optimize Tailwind CSS bundle (Tailwind v4) Add explicit @source directive for static scanning and @layer utilities safelist for runtime-constructed risk/severity colour classes. - MettaChain#117: Extend audit logging for all security events Add logAuthFailure, logSettingsChange, logKycStatusChange, logTransactionInitiation, and logTransactionCompletion methods to SecurityAuditLogger, covering all events listed in the issue. - MettaChain#116: Add React Suspense loading states Create PropertyPageSkeleton component and loading.tsx files for /properties, /properties/[id], and /dashboard routes. Error boundaries were already present via error.tsx files.
|
@DanielCharis1 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits. You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Fixes all four issues assigned to DanielCharis1 in a single PR.
#121 – Docs: Document smart contract integration points
Added
docs/smart-contract-integration.mdcovering:#118 – Optimize Tailwind CSS bundle with PurgeCSS
Tailwind v4 uses CSS-based config (no
tailwind.config.js). Content scanning is built-in, but two gaps were addressed insrc/app/globals.css:@sourcedirective so the scanner covers allsrc/**/*.{ts,tsx}files.@layer utilitiessafelist for runtime-constructed risk/severity colour classes (.text-risk-*,.bg-risk-*) that the static scanner cannot detect.No visual regressions – only new utility classes were added.
#117 – Add comprehensive audit logging for security events
Extended
SecurityAuditLoggerinsrc/utils/security/auditLogger.tswith five new methods:logAuthFailureauth_failurelogSettingsChangesettings_changelogKycStatusChangekyc_status_changelogTransactionInitiationtransaction_initiationlogTransactionCompletiontransaction_completionAll five event types were also added to the
AuditLogEntry.eventTypeunion.#116 – Implement proper loading states with React Suspense
src/components/PropertyPageSkeleton.tsx– skeleton that matches the properties page layout (already imported by/properties/page.tsx).loading.tsxfiles for three routes (Next.js App Router automatic Suspense):src/app/properties/loading.tsxsrc/app/properties/[id]/loading.tsxsrc/app/dashboard/loading.tsxerror.tsxfiles on each route.Testing
npx tsc --noEmit– pre-existing errors are unrelated to this PR).Closes #116
closes #117
closes #118
closes #121