Skip to content

Front End: CRUD profiles, badges, attestations#24

Merged
oscarwroche merged 19 commits intomainfrom
create-profile
Sep 16, 2025
Merged

Front End: CRUD profiles, badges, attestations#24
oscarwroche merged 19 commits intomainfrom
create-profile

Conversation

@joelamouche
Copy link
Contributor

@joelamouche joelamouche commented Sep 12, 2025

Backend

  • add get all profiles endpoint
  • add readme

Front end

Profiles

  • add create profile logic and components
  • add update profile logic and components
  • add delete profile logic and components
  • add profile page

Badges

  • add create and get badges logic and components

Attestations

  • add create and get attestations logic and components

Smart contracts

  • register attestation ids in the activity token in order to fetch them in the fe

handles #18
handles #25

@joelamouche joelamouche marked this pull request as draft September 12, 2025 15:38
@joelamouche joelamouche changed the title Create profile Front End: CRUD profiles Sep 12, 2025
@joelamouche joelamouche changed the title Front End: CRUD profiles Front End: CRUD profiles, badges, attestations Sep 15, 2025
@joelamouche joelamouche marked this pull request as ready for review September 16, 2025 10:50
@joelamouche joelamouche enabled auto-merge (rebase) September 16, 2025 10:50
use crate::domain::repositories::profile_repository::ProfileRepository;
use std::sync::Arc;

pub async fn get_all_profiles(
Copy link
Collaborator

Choose a reason for hiding this comment

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

👍👍👍

.route("/profiles/:address", delete(delete_profile_handler));
.route("/profiles/:address", delete(delete_profile_handler))
.with_state(state.clone())
.layer(from_fn_with_state(state.clone(), eth_auth_layer));
Copy link
Collaborator

Choose a reason for hiding this comment

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

👍

auto-merge was automatically disabled September 16, 2025 10:53

Rebase failed

@oscarwroche oscarwroche merged commit a14be82 into main Sep 16, 2025
2 of 3 checks passed
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

Comments