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

Basic agreement functionality #30

Open
wants to merge 12 commits into
base: kaiwen_dev
Choose a base branch
from

Conversation

biankaiwen111
Copy link
Member

No description provided.

{ label: "Labor Studies/B.A.", value: "Labor Studies/B.A." },
{ label: "Global Studies/B.A.", value: "Global Studies/B.A." },
{ label: "Psychobiology/B.S.", value: "Psychobiology/B.S." },
];

Choose a reason for hiding this comment

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

It would be better if the frontend and backend can share the same university/cc/major list.

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, this part will be store in our db in the future, and will be requested from db when user select the target university and cc. The dropdown menu for majors will be render dynamiclu based on that.

Copy link
Member

Choose a reason for hiding this comment

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

Shall we open a new repo called AssistFetch to fetch uni/cc/major list from assist.org?

Choose a reason for hiding this comment

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

This sounds great.


const searchHandler = (event) => {
console.log(university, cc, major);
//fetch Data from DB, suppose we have data here

Choose a reason for hiding this comment

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

// Todo(Kevin): fetch Data from DB, suppose we have data here

Copy link
Member

Choose a reason for hiding this comment

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

@biankaiwen111 Can you manually input those data into db and make adjustment to the code here? So we don't have to have this example.is file.

After that, you can start writing APIs for inputting the data and then refine the data display method.

The step after that should then be finding a good method/visualization for volunteer inputting

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.

3 participants