pantheon/hestia
2
0
Fork 0
Code Issues 23 Pull requests Projects Releases Packages Wiki Activity Actions

Residents frontend page #82

Merged
BenjaminPalko merged 8 commits from 79-residents-frontend-page into master 2025-02-11 17:15:20 -05:00
Conversation 2 Commits 8 Files changed 19 +415 -39
BenjaminPalko commented 2025-02-08 20:43:35 -05:00 (Migrated from github.com)
Copy link

closes #79

image
image

closes #79 ![image](https://github.com/user-attachments/assets/7f43a3ea-8fd6-4f8b-9011-1e37658fbc72) ![image](https://github.com/user-attachments/assets/e66f7537-f6ce-447a-9f0e-48d052288fe2)
DanMihailescu (Migrated from github.com) reviewed 2025-02-08 20:43:35 -05:00
netlify[bot] commented 2025-02-08 20:43:50 -05:00 (Migrated from github.com)
Copy link

Deploy Preview for hestia-home ready!

Name Link
Latest commit a77e75d115
Latest deploy log https://app.netlify.com/sites/hestia-home/deploys/67ab5ced4a96860008f36061
Deploy Preview https://deploy-preview-82--hestia-home.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 site configuration.

### <span aria-hidden="true">✅</span> Deploy Preview for *hestia-home* ready! | Name | Link | |:-:|------------------------| |<span aria-hidden="true">🔨</span> Latest commit | a77e75d11591435d28481cff882bea80ea8f784c | |<span aria-hidden="true">🔍</span> Latest deploy log | https://app.netlify.com/sites/hestia-home/deploys/67ab5ced4a96860008f36061 | |<span aria-hidden="true">😎</span> Deploy Preview | [https://deploy-preview-82--hestia-home.netlify.app](https://deploy-preview-82--hestia-home.netlify.app) | |<span aria-hidden="true">📱</span> Preview on mobile | <details><summary> Toggle QR Code... </summary><br /><br />![QR Code](https://app.netlify.com/qr-code/eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJ1cmwiOiJodHRwczovL2RlcGxveS1wcmV2aWV3LTgyLS1oZXN0aWEtaG9tZS5uZXRsaWZ5LmFwcCJ9.e1ywscptjDAXY-FHMi5O4p3waluYAUu3G_CvaRRiG8I)<br /><br />_Use your smartphone camera to open QR code link._</details> | --- <!-- [hestia-home Preview](https://deploy-preview-82--hestia-home.netlify.app) --> _To edit notification comments on pull requests, go to your [Netlify site configuration](https://app.netlify.com/sites/hestia-home/configuration/notifications#deploy-webhooks)._
BenjaminPalko commented 2025-02-10 15:48:52 -05:00 (Migrated from github.com)
Copy link

Ive just added editing for residents:
image
image

Ive just added editing for residents: ![image](https://github.com/user-attachments/assets/167419fd-d71d-41a6-80c0-32238c6b7d66) ![image](https://github.com/user-attachments/assets/723c2e0d-86b2-4ed8-83f2-8efd60691b37)
piopi (Migrated from github.com) approved these changes 2025-02-10 19:29:46 -05:00
piopi (Migrated from github.com) left a comment
Copy link

Very nice!

Very nice!
src/routes/app/residents/+page.server.ts
@ -0,0 +26,4 @@
const form = await event.request.formData();
if (!form.has('name')) {
return fail(400, { error: 'phone_missing' });
piopi (Migrated from github.com) commented 2025-02-10 19:29:16 -05:00
Copy link

Errors could be an enum, to be used by the frontend

Errors could be an enum, to be used by the frontend
BenjaminPalko (Migrated from github.com) reviewed 2025-02-11 17:15:07 -05:00
src/routes/app/residents/+page.server.ts
@ -0,0 +26,4 @@
const form = await event.request.formData();
if (!form.has('name')) {
return fail(400, { error: 'phone_missing' });
BenjaminPalko (Migrated from github.com) commented 2025-02-11 17:15:07 -05:00
Copy link

I'd like to work out a system with you guys for handling these

I'd like to work out a system with you guys for handling these
Sign in to join this conversation.
Reviewers
No reviewers
DanMihailescu
piopi
Labels
Clear labels   
bug

Something isn't working

  
documentation

Improvements or additions to documentation

  
duplicate

This issue or pull request already exists

  
enhancement

New feature or request

  
good first issue

Good for newcomers

  
help wanted

Extra attention is needed

  
invalid

This doesn't seem right

  
question

Further information is requested

  
refactor

Rework something

  
wontfix

This will not be worked on

No labels
bug
documentation
duplicate
enhancement
good first issue
help wanted
invalid
question
refactor
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: pantheon/hestia#82
No description provided.
Delete branch "79-residents-frontend-page"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?