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

Added buttons for syncing #414

Merged

Conversation

MayurLund
Copy link
Contributor

@MayurLund MayurLund commented Jul 19, 2023

Issue #387

Overview:
Added 3 buttons for syncing the data for discord users data and unverified Discord users data. The buttons are shown only to super user.

screen-capture (2)

The others two buttons are failing as I don't have discord setup but I have written the logic for it and its working fine in colleague local system who is having the whole setup and dummy database.

The test cases are yet to be written for the above work.

index.html Show resolved Hide resolved
@gauravsinhaweb
Copy link

@MayurLund It would be nice if you could attach a working video of the implementation, and also provide a short description in the PR of what changes/ functionality you made in this PR

style.css Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
script.js Outdated Show resolved Hide resolved
script.js Outdated Show resolved Hide resolved
script.js Outdated Show resolved Hide resolved
script.js Outdated Show resolved Hide resolved
script.js Outdated Show resolved Hide resolved
style.css Show resolved Hide resolved
script.js Outdated Show resolved Hide resolved
sahsisunny
sahsisunny previously approved these changes Jul 21, 2023
Copy link
Member

@sahsisunny sahsisunny left a comment

Choose a reason for hiding this comment

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

Please link the ticket for test

heyrandhir
heyrandhir previously approved these changes Jul 22, 2023
@MayurLund
Copy link
Contributor Author

Please link the ticket for test
Created a new issue #422 for tests and linked up with the PR

@MayurLund MayurLund dismissed stale reviews from heyrandhir and sahsisunny via 5c3e4b6 July 23, 2023 15:33
@prakashchoudhary07 prakashchoudhary07 merged commit 93881ed into Real-Dev-Squad:develop Jul 23, 2023
4 checks passed
@MayurLund MayurLund mentioned this pull request Jul 25, 2023
12 tasks
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.

5 participants