Recent Q/A Sessions in Q/A Tab #20
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Owner
yashsehgal
commented
Oct 10, 2023
•
yashsehgal
commented
- closes [Task]: Add cards for recent sessions #21
- closes [Task]: Create a empty state for sessions page when no recent sessions are available #19
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
3 tasks
…r and imported it in page.tsx, changed the values of sessionID in qna-session-mock.ts to make all of them unique
…rning array of interface to object of type array of interfaces (reference: FeatureList object from dashboard-features-list)
yashsehgal
referenced
this pull request
Oct 11, 2023
|
The vercel deployment is failing @shristigupta12 Make sure you do |
Changes required
Here, do not use another fetch import for getting recents data. Instead, send this state as a prop. Lines 26 to 29 in e8d7005
Other than these changes, good work. After this, we can merge this with cc: @shristigupta12 ❤️ |
yashsehgal
changed the title
yashsehgal
added a commit
that referenced
this pull request
Oct 12, 2023
* feat: dashboard sections boilerplate (#9) * feat: created view container * feat: created navbar for review with some fixes to be addressed * fix: view-container type / merge conflict * style: view container breakpoints * fix(style): remove bg-red for testing * style: tracking-tight for headings * style: theme switch upgrade * feat: page container layout, feature options added * chore: add icons in features list data * style: feature list responsive layout * style: test logo in navbar * Recent Q/A Sessions in Q/A Tab (#20) * fix: add link support for option type * feat: flow for new qna session * style: sessions page header layout * chore: add env file to .gitignore * chore: add environment variables to shared commons * feat: add methods for handling data requests, with mock data support * feat: add UI for empty state, recent sessions view * fix: created a new recent-qna-session-list component in section folder and imported it in page.tsx, changed the values of sessionID in qna-session-mock.ts to make all of them unique * fix: changed the data of qna-session-mocks.ts file from function returning array of interface to object of type array of interfaces (reference: FeatureList object from dashboard-features-list) * feat: a rough design of the recent-qna-sessions * fix-styles: fixed a hover brightness style * fix style: created a common class box-card to reduce css repitition * feat: add component for listing items in flex * fix: vercel key-prop issue resolved * fix: recent sessions list rendering * chore: feature list data shifted to common * feat: created a box-card component in layout * chore: changed name component-list to box-card-slider * chore: file name changed and box-card style removed * fix: build error fixed * fix: build error fixed * chore: used usestate as prop in recent-qna-sessions-list * fix: prop error fix in recent-qna-session * fix: Style changes * style: update text size in recent q/a session cards * feat: add prettier config * chore: prettier fix * style: remove ring on links and buttons * style: add dialog content bg * feat: mock data manager for topics & depth --------- Co-authored-by: Shristi Gupta <shristiguptawork@gmail.com> --------- Co-authored-by: Shristi Gupta <shristiguptawork@gmail.com>
This was referenced Oct 12, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.