Se agregan los campos createdAt, updatedAt, requeridos por la API RES…#20
Se agregan los campos createdAt, updatedAt, requeridos por la API RES…#20fdanielpm wants to merge 1 commit into
Conversation
📝 WalkthroughWalkthroughThe ChangesViewer Timestamp Properties
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (1)
src/main/java/com/mifiel/api/objects/Viewer.java (1)
77-91: ⚡ Quick winInconsistent indentation detected.
The new getter and setter methods use tab characters for indentation, while the rest of the file uses spaces (4 spaces per indent level). Please update the indentation to maintain consistency with the existing code style.
📐 Suggested fix for indentation consistency
Replace tabs with 4 spaces in the new methods:
- public String getCreatedAt() { - return createdAt; - } + public String getCreatedAt() { + return createdAt; + } - public void setCreatedAt(String createdAt) { - this.createdAt = createdAt; - } + public void setCreatedAt(String createdAt) { + this.createdAt = createdAt; + } - public String getUpdatedAt() { - return updatedAt; - } + public String getUpdatedAt() { + return updatedAt; + } - public void setUpdatedAt(String updatedAt) { - this.updatedAt = updatedAt; - } + public void setUpdatedAt(String updatedAt) { + this.updatedAt = updatedAt; + }🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@src/main/java/com/mifiel/api/objects/Viewer.java` around lines 77 - 91, The new getter/setter methods in class Viewer (getCreatedAt, setCreatedAt, getUpdatedAt, setUpdatedAt) use tab characters for indentation; replace those tabs with four space characters per indent level so they match the file's existing 4-space indentation style and maintain consistency across the class.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Nitpick comments:
In `@src/main/java/com/mifiel/api/objects/Viewer.java`:
- Around line 77-91: The new getter/setter methods in class Viewer
(getCreatedAt, setCreatedAt, getUpdatedAt, setUpdatedAt) use tab characters for
indentation; replace those tabs with four space characters per indent level so
they match the file's existing 4-space indentation style and maintain
consistency across the class.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 55b0119a-6a24-4905-ad0a-6a0e7f5b65b6
📒 Files selected for processing (1)
src/main/java/com/mifiel/api/objects/Viewer.java
Se agregan los campos createdAt, updatedAt, requeridos por la API REST/JSON al objeto com.mifiel.api.objects.Viewer, sin éstos campos se genera un error, al crear un documento o listarlo, cuando se han agregado espectadores a la firma del documento.
Summary by CodeRabbit