Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update admin-site to support json schema #1180

Merged
merged 1 commit into from Oct 26, 2023
Merged

Conversation

norbye
Copy link
Member

@norbye norbye commented Jul 24, 2023

For the main backend changes, see #1178.

Updates the admin-views to support viewing applications, download CSVs and edit the JSON schema.

The backend + the frontend structure supports adding fields, reordering fields, etc. but this is not implemented in the GUI yet. Would rather do this in a later PR.

The functionality to change the top questions (info and phone number in the screenshot in #1179) is not yet implemented, but will match the visual preview for group-leaders.

Visual preview

Admin page for group-leaders and recruiters

image image

Admin page for admission-admins

image

Resolves ABA-515, ABA-516

@norbye norbye added the review-needed Pull requests that need review label Jul 24, 2023
@norbye norbye requested a review from a team July 24, 2023 05:59
Copy link
Contributor

@PeterJFB PeterJFB left a comment

Choose a reason for hiding this comment

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

First look-through looks nice 💯
Left a few comments but nothing critical considering it being an mvp

frontend/src/utils/methods.ts Show resolved Hide resolved
frontend/src/utils/methods.ts Outdated Show resolved Hide resolved
Comment on lines +17 to +24
name={"questions." + index + ".name"}
placeholder={""}
label={"Label"}
/>
<Field
name={"questions." + index + ".label"}
component={TextAreaField}
placeholder={""}
Copy link
Contributor

Choose a reason for hiding this comment

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

Looks like JsonFieldTextAreas placeholder-field is supposed to be used here 🤔

@norbye norbye marked this pull request as draft August 22, 2023 08:25
@norbye
Copy link
Member Author

norbye commented Aug 22, 2023

Converted to draft to ensure it is not merged before 2023 admissions, will merge some time after that.

@norbye norbye added approved Pull requests that have been approved and removed review-needed Pull requests that need review labels Aug 22, 2023
@norbye norbye marked this pull request as ready for review October 26, 2023 23:30
@norbye norbye merged commit 901ed57 into json-schema Oct 26, 2023
2 of 3 checks passed
@norbye norbye deleted the json-schema-admin branch October 26, 2023 23:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Pull requests that have been approved
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants