-
Notifications
You must be signed in to change notification settings - Fork 23
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
chore: refactor fetching functions #620
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
6ad7881
to
6b61d08
Compare
6b61d08
to
a0d7af6
Compare
a0d7af6
to
242c694
Compare
Instead of request, shouldn't it be "proposal"? I think request is not clear for users. Also, on the glossary we talk about proposals |
Since it's So maybe, we could name it like this: how do you think? @ctrlc03 @crisgarner |
242c694
to
aea5e0f
Compare
aea5e0f
to
d20a7f1
Compare
I think the changes on the API and code make sense, as you mentioned, it's named request, on the UI to the user I would leave it as Proposal, and the proposal calls the add request. The only thing I'm not convinced is with "Project" as in theory, not all proposals will be projects, some times will be yes or no questions. |
I think it makes sense, though I agree with Cris "concern" on the term Project, which might not always apply. What about Option? |
d20a7f1
to
5dcc3e8
Compare
@crisgarner @ctrlc03 And I'm not sure about the |
I like the idea, in the self-service, depending on which type of round you are creating the name will change. |
I would open an issue for it, let's keep this PR focusing on the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks
Description
The name of
Application
andProject
is confusing, meanwhile there areRequest
andRecipient
on the contract, wanna make the name mapped well. Also, there are several functions or interfaces not being used, should be removed.Application
toRequest
Related issue(s)
close #545
Confirmation
Important
We do not accept minor grammatical fixes (e.g., correcting typos, rewording sentences) unless they significantly improve clarity in technical documentation. These contributions, while appreciated, are not a priority for merging. If there is a grammatical error feel free to message the team.