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.

Sign up for GitHub

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

Jump to bottom

Eng 245 Do not show interactive UI #336

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

Eng 245 Do not show interactive UI #336

wants to merge 8 commits into from

Conversation

Parikshit85
Copy link
Contributor

@Parikshit85 Parikshit85 commented May 1, 2024 •
edited
Loading

Pull Request

Description

  • Interactive UI should not be shown each time the user visits the particular page.
  • Maintain a flag at the backend to store if the user guide is shown to the user or not

Checklist

  • I have tested the changes locally and they are functioning as expected.
  • I have assigned the PR to the appropriate reviewer(s) for their feedback.
  • Are all the improvements/development done?

@Parikshit85 Parikshit85 added enhancement New feature or request draft draft: in progress do not merge labels May 1, 2024
@Parikshit85 Parikshit85 requested a review from varsha1305nav May 1, 2024 08:07
@Parikshit85 Parikshit85 self-assigned this May 1, 2024
@linear Linear
Copy link

linear bot commented May 1, 2024

ENG-245 Interactive UI should be shown only first time

Interactive UI should not be shown each time the user visits the particular page.

Maintain a flag at the backend to store if the user guide is shown to the user or not

Note:- Come up with a data model and discuss with me.

@Parikshit85 Parikshit85 changed the title Draft: Eng 245 Do not show interactive UI Eng 245 Do not show interactive UI May 7, 2024
@Parikshit85 Parikshit85 removed the draft draft: in progress do not merge label May 7, 2024
@muditmahajan muditmahajan added the stacked This is a stacked PR. Please merge the other dependent PR first. label Jun 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@varsha1305nav varsha1305nav Awaiting requested review from varsha1305nav

Assignees

@Parikshit85 Parikshit85

Labels
enhancement New feature or request stacked This is a stacked PR. Please merge the other dependent PR first.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Parikshit85 @muditmahajan