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

Survey notification implementation #1035

Merged
merged 9 commits into from
Jun 17, 2022
Merged

Conversation

francescospissu
Copy link
Contributor

Motivation

A survey is needed to investigate the motivation, ability and context of use of the IDE 2 by users.

Change description

A notification box has been implemented with a button to redirect to the survey and another to reject it.

Reviewer checklist

  • PR addresses a single concern.
  • The PR has no duplicates (please search among the Pull Requests before creating one)
  • PR title and description are properly filled.
  • Docs have been added / updated (for bug fixes / features)

@per1234 per1234 added type: enhancement Proposed improvement topic: code Related to content of the project itself labels Jun 11, 2022
@francescospissu francescospissu marked this pull request as draft June 13, 2022 09:35
@francescospissu francescospissu force-pushed the fspissu/survey-notification branch from 52c1221 to 61b8b8c Compare June 13, 2022 16:06
@francescospissu francescospissu marked this pull request as ready for review June 14, 2022 12:34
@per1234 per1234 dismissed their stale review June 14, 2022 14:31

Requested changes have been made. Thanks!

Copy link
Contributor

@kittaakos kittaakos left a comment

Choose a reason for hiding this comment

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

Very nice code 😍 Thank you!

@francescospissu francescospissu force-pushed the fspissu/survey-notification branch from 226f0b8 to 77698f8 Compare June 16, 2022 16:30
@francescospissu francescospissu merged commit f5cee97 into main Jun 17, 2022
@francescospissu francescospissu deleted the fspissu/survey-notification branch June 17, 2022 08:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: code Related to content of the project itself type: enhancement Proposed improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants