-
-
Notifications
You must be signed in to change notification settings - Fork 170
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
Feature/Handle Loading And Empty Data Message #464
Merged
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
- What: Add spinner when the 'Users', 'Blogs', 'CFT' components are loading - Why: Better user experience - How: Check the loading state property of each data and act accordingly
-What: Notify users by message that an object data is not found of components in Explore Route -How: Make NotFound component get title and description by props -Why: Better User experience
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.
It's great having you contribute to this project
🌟 Welcome to the community 🌟
- If you would like to continue contributing to open source and would like to do it with an awesome inclusive community.
- You should join our Discord chat and our GitHub Organisation.
- We help and encourage each other to contribute to open source little and often 😄.
- Any questions let us know.
kabir0x23
added a commit
that referenced
this pull request
Jun 9, 2024
* fixed #451 (#454) * fixed #451 * Update package-lock.json Co-authored-by: Kabir <[email protected]> * Update package.json * fix: package.json --------- Co-authored-by: Kabir <[email protected]> * fix: icons over flow in footer (#460) * fix: icons over flow in footer * fix icon over flow in footer * [FEATURE] Settings Edit Main Profile Page #448 (#457) * fix: Message icon is in middle in mobile version issue - #443 * feat: Settings Edit Main Profile Page - issue #448 * Created note taking functionality in dashboard (#463) * You can now create note, delete note * styling added improvements to be done --------- Co-authored-by: Kabewh <[email protected]> * Feature/Handle Loading And Empty Data Message (#464) * Add Loading Spinner For Components In Explore Route - What: Add spinner when the 'Users', 'Blogs', 'CFT' components are loading - Why: Better user experience - How: Check the loading state property of each data and act accordingly * Add Data Not Found message -What: Notify users by message that an object data is not found of components in Explore Route -How: Make NotFound component get title and description by props -Why: Better User experience * fix: minor issues (#468) * fix: upgrade react-router from 6.4.2 to 6.15.0 (#456) Snyk has created this PR to upgrade react-router from 6.4.2 to 6.15.0. See this package in npm: https://www.npmjs.com/package/react-router See this project in Snyk: https://app.snyk.io/org/kabir0x23/project/3518e29a-eaff-41d1-8ccf-a14e65eabf3c?utm_source=github&utm_medium=referral&page=upgrade-pr Co-authored-by: snyk-bot <[email protected]> * fix: upgrade react-router-dom from 6.4.2 to 6.16.0 (#465) Snyk has created this PR to upgrade react-router-dom from 6.4.2 to 6.16.0. See this package in npm: https://www.npmjs.com/package/react-router-dom See this project in Snyk: https://app.snyk.io/org/kabir0x23/project/3518e29a-eaff-41d1-8ccf-a14e65eabf3c?utm_source=github&utm_medium=referral&page=upgrade-pr Co-authored-by: snyk-bot <[email protected]> * fix: minor issues --------- Co-authored-by: snyk-bot <[email protected]> * fix: ReadMe.md (#484) * fix: ReadMe.md * Update README.md * feat: Improve Dashbaord SideNavBar Design, Closes #472 (#481) * feature: sidenav bar redesign * feat: user data --------- Co-authored-by: Kabir <[email protected]> * feat: option to share certificate on socials. (#479) * added feature to share on socials * fix: minor issues --------- Co-authored-by: Kabir <[email protected]> * feat: Leaderboard design Improvement (#470) * ui fixed with first leaderboard user to be highlighted * fix: When Menu Is Open Fix Scrollable Background (#488) What: fix scrollable background when menu is open on mobile display How: add overflow hidden when menu is open so the bg will not be Scrollable Why: better user experience * feat: chatbot (#489) * fix: Chatbot (#490) * feat: chatbot * feat: chatbot * fix: chatbot comments * fix: chatbot comments * feat: quiz-new version updated (#495) * fix: communinty links padding and font-size * added 3 styled components and arraged hover and alignment * used newly added styled compoenents * removed repetative styled compoenet * replaced LeaderboardTableHeaderUsers with LeaderboardTableHeader * replaced the old crown icon with trophy icon * replaced the old logo with new trophy logo * replaced crown icon with trophy * replaced crown icon with trophy * added 2 styled spans for managing ui * ui fixed with first leaderboard user to be highlighted * added breakpoint for cards to appear in center * fix: communinty links padding and font-size --------- Co-authored-by: Kabir <[email protected]> * feat: Timeline/Events page implementation #471 (#499) * feature: sidenav bar redesign * feat: user data * feat: timeline events logic * feat: feature timeline events * add: event description --------- Co-authored-by: Kabir <[email protected]> * fix: feed post design (#503) * Update README.md (#505) * feat: community chat frontend (#504) * fix: alignement of icon and text in the navbar (#486) * chore(release): v0.53.15 [skip ci] * chat area frontend added * Chat page ui enhanced * SendMessage area added * fix: minor issues * Update CHANGELOG.md --------- Co-authored-by: Mario Randrianomearisoa <[email protected]> Co-authored-by: Conventional Changelog Action <[email protected]> Co-authored-by: Kabir <[email protected]> * fix: dashboard minor issues (#506) * feat: sponsors page (#508) * fix: alignement of icon and text in the navbar (#486) * chore(release): v0.53.15 [skip ci] * sponsors section added * fix: version * Update CHANGELOG.md * fix: design and add button * fix: design and add button * fix: design and add button * fix: design and add button --------- Co-authored-by: Mario Randrianomearisoa <[email protected]> Co-authored-by: Conventional Changelog Action <[email protected]> Co-authored-by: Kabir <[email protected]> * feat: notes taking app - base (#524) * [Feat] Add NoteApp Layout How: with styles Why: according to the design sample * [Feat] Add Layout for NotesSidebarHeader and NotesSidebarSearch How: html/css in react Why: progressing * [Feat] Isolate SearchInputBox component Why: so we could reuse it How: Isolate SearchInputBox to different file and use in right places * [Feat] Change the Toggle Button of Sidebar Position Why: the prev position get on elements in the main container How: change some styles * [Feat] Add NotesList in SideBar What: add notes list in SideBar How: create components for list, item style , pass data Why: need to implement * [Feat] Add Search Functionality, Add Pick Note And Show Description why: to be able to search notes. to show description of notes how: react... * [Feat] Add Search Functionality, Add Pick Note And Show Description why: to be able to search notes. to show description of notes how: react... * feat: notes taking app - pin and delete functionality (#526) * [Fix] React does not recognize the isOpen prop on a DOM element why: it needed to be fixed how: don't pass the isOpen to the component * [Feat] Add Pin and Delete Functionality why: so the user could pin and delete note how: when user delete this note removes, when pin the pinned value changes and styles * feat: reverse shell gen (#525) * feat: Note App Markdown Editor (#527) * [Feat] Add MarkdownEditor Component why: so we could add and edit markdown text how: with react-md-editor * [Feat] Add MDEditor Label , Change Some Styles how: change styles as suggested in the docs of react-md-editor why: looks better * [Feat] Add Cancel Btn , Handle Note Without Title/Descr how: when click on cancel we go from edit/add mode without change. put some place holder for this data. why: so the user could close mdeditor. so the user will see something instead * [Feat] Change Some Func Names Why: make it clearer * [Feat] Add InputEditor For Title why: we dont need markdown editor for Title how: create new component that do it * [Feat] Change InputEditor Styles why: make it clearer that this is input field * feat: Hackbook (#528) * feat: hackbook | base - 1 * feat: Hackbook | base - 2 * feat: HackBook * feat: feed tags * fix: minor issues (#529) * [Bug] Fixing Import Source of SearchBox, SearchIcon, SearchInput (#530) why: those elements are from "../Common/SearchInputBox/SearchInputBoxElements"; * fix: package-lock.json file * feat: Note App Add/Edit Notes , Managing State with Redux (#531) * [Bug] Fixing Import Source of SearchBox, SearchIcon, SearchInput why: those elements are from "../Common/SearchInputBox/SearchInputBoxElements"; * [Feat] Manage State In Redux and Implement Add/Edit Note why: better to manage state with redux how: redux-toolkit slice... * [Fix] Remove old unused code why: we dont use this code anymore * [Fix] Remove Note Reference When Create New Note why: so the prev note data will not copied to current how: setPickedNote to empty obj * [Fix] Remove Markdown Symbols From Description In Note List why: to make it cleaner how: with regexp * fix: user and blog routes (#533) * feat: notes | Connect Notes to Backend (#534) * [Feat] Connect Notes To Backend why: so the notes will be stored in db how: change the redux slice file, add fetching data with axios * [Feat] Connect Notes To Backend why: so the notes will be stored in db how: change the redux slice file, add fetching data with axios * fix: notes | minor bugs (#535) * [Feat] Connect Notes To Backend why: so the notes will be stored in db how: change the redux slice file, add fetching data with axios * [Feat] Connect Notes To Backend why: so the notes will be stored in db how: change the redux slice file, add fetching data with axios * [Fix] Search Also By content why: find searchable values that are in the content property how: change from description to content because the property is content * [Fix] Untitled Notes Change Shown Part From Id why: the first 5 digits of id is similar to one another how: show 10 last digits * [Feat] Close Create Note When Picking a Note why: so move from create note to picking note if the user want it how: close create mode when user click a diffrent note * [Fix] Close Edit Mode When Picking a Different Note why: better ux how: close edit mode when picked note changes * [Fix] Show Note Before Changes When Closing Edit Mode why: because the user when click on close btn dosent want to save the Changes how: show prev note before changes * feat: notes | make checkboxes clickable (#536) * [Feat] Connect Notes To Backend why: so the notes will be stored in db how: change the redux slice file, add fetching data with axios * [Feat] Connect Notes To Backend why: so the notes will be stored in db how: change the redux slice file, add fetching data with axios * [Feat] Make the CheckBoxes Clickable why: better ux how: add plugin and listen to changes * [Fix] Remove unwanted console.log * fix: user | log-out on token exp (#542) * fix: user | log-out on sessoin expire * feat: notes | Func of Paste, Drop Image In editor (#541) * [Feat] Create new CheckBoxClickable Component why: make the MarkdownEditor Component cleaner how: move relevant code to CheckBoxClickable Component file * [Feat] Add Func of Paste, Drop Image In editor why: so user could paste and drop images in md editor how: integrate the code from CreateBlog * [Feat] Manage the Image Path to be Correct why: so we could also display the Image how: as was done in Feeds page * [Bug] Not Related, Fix AutoComplete Warning In Login Page why: it appers non stop how: set "off" instead of false or null * [Feat] Update The Description Component State When Add Images why: so when the user save the note the images will be added to Description context how: pass the handleChange to useImageUploadEvents and update it * [Feat] Some Tweeks What: 1. change "undescribedNote" to "(Empty)" 2. remove id from "untitledNote" to "untitledNote #number" 3. remove all the address of the image and leave only the name of image how: 1. change when needed 2. make count in redux 3. with regexp why: 1. looks better 2. to not show the id of note to user 3. looks better * [Feat] Some Style Tweaks what: 1. change the title editor , dont need to preview what we input 2. make description and description preview in edit mode scrollable 3. handle text overflow when needed why: better ux how: 1. change styles 2. overflow-y:auto and limit width 3. overflow-wrap:break-word; * feat: notes | some more tweaks (#543) * [Fix] Skip Line When Add Image why: so the text will be under and not beside the image how: "\n\n" add to image text * [Fix] Remove String Comparison Helper Function why: string is primitive data type so we can compare with "===" how: use "===" instead * [Feat] Make Indection That the Note Was Picked why: better ux how: this note's position changes a little bit * [Feat] Change SearchInputBox Func & Styles why: so when we click on icon we focus inside, style icon inside how: with focus() and blur() * [Feat] Limit Title For Picked Note in List Of Notes why: so the title will not be under the pin icon how: limit to 25 char * [Fix] Make Better Indication For Picked Note why: looks better how: change some styles * feat: dasboard | design improvement (#544) * feat: notes | new design (#545) * feat: community chat (#546) * feat: community chat | send messages * feat: community chat | send messages * feat: community chat | auto scroll * feat: community chat | fetch messages * feat: community chat | fetch messages * feat: community chat | design improvement * feat: community chat | fix api url (#547) * feat: community chat | send messages * feat: community chat | send messages * feat: community chat | auto scroll * feat: community chat | fetch messages * feat: community chat | fetch messages * feat: community chat | design improvement * feat: community chat | fix backend api * fix: twitter logo (#540) * Twitter Logo Changed * Twitter logo change in certificate and public profile * fix package-lock.json file * fix package-lock.json file * fix package-lock.json file --------- Co-authored-by: Kabir <[email protected]> * feat: community chat | fix connection issue * feat: community chat | fix connection issues - 1 * feat: community chat | fix connection issues - 1 * feat: community chat | fix connection issues - 1 * feat: community chat | fix connection issue * feat: community chat | fix api path * fix: notes | hide title description and pick first note by default #548 why: 1. when none note is picked note display bg of descr title 2. when reload page or going first time to notes page pick first note by default how: 1. hide when note was not picked 2. after dispatch pick first note * feat: notes | categories base (#549) * [Fix] Hide Description Title And Pick First Note By Default why: 1. when none note is picked note display bg of descr title 2. when reload page or going first time to notes page pick first note by default how: 1. hide when note was not picked 2. after dispatch pick first note * [Feat] Add List Of Categories and Make Note Category Be Picked Category why: so the user could use Categories how: add list of categories, when creating new note make it's category to be as the picked category from where we created it * Update NoteApp.jsx --------- Co-authored-by: Kabir <[email protected]> * feat: notes | categories create/edit/remove (#551) * [Fix] Change Title to "Notes" and Fill Icons With Color why: Notes title better. The icons should be filled how: chage title . add fill prop to "path" element inside the class element * [Feat] Add Functionality Of Creating New Category Why: so the user could create new category How: manage with local state with redux * [Feat] Add Functionality of Remove/Edit Category Why: so user could remove and edit category as well How: manage state with redux * feat: notes | category connect to backend (#553) * [Feat] Modify Redux To Be Related To Server why: the data in the server and manage the local state with redux how: createAsyncThunk and axios * [Feat] Refactoring NoteSidebar To Be Separate component why: make the code cleaner how: react * [Feat] Use Modified Redux Where Needed why: make sure we send the right data to redux functions how: check manually what data need to be passed * feat: cheatsheets & tools | pass gen, rev shell (#554) * feat: community chat | user chat | design improvement * feat: tools | shell generator * feat: cheatsheets && tools | passgen * feat: cheatsheets && tools | passgen * feat: cheatsheets && tools | passgen * feat: ai chat app (#555) * feat: ai chat app (#556) * feat: ai chat app * feat: ai chat app * fix: ai chat app | minor issues (#557) * feat: ai chat app * feat: ai chat app * feat: notes | Change the CRUD Category UI, Leave One Required Category #558 Why: better UI, one required category is enough How: instead of options mode , make the modifications in place, options menu open with three dots * feat: community events | improvement (#569) * [Feat] Create New Dashboard For Admins Why: some features only admin can manage How: as Dashboard , shared component * [Feat] The CommunityEvents Base , Refactoring The Prev Code Why: so we could use same component in main page and in the admin Dashboard How: use shared component * fix: scrollbar buttons are removed (#565) * Scrollbar buttons are removed * Duplicated selector removed * [Feat] Protect The AdminDashboard Route (#570) Why: only userType of "admin" or "team" can access the AdminDashboard How: render this tab only for right users, redirect for other users * [Feat] Make The Displayed CommunityEvents Functional (#572) How: using react and temporary storing the events that was picked in some state in redux later will change in right obj data Why: so the user could join and see the status of it in the page * New course UI (#586) * feat: Quiz Routes * feat: Quiz Route * feat: NewCourses * feat: Quiz Routes (#585) * feat: Quiz Routes * feat: Quiz Route * fix: feeds search bug fixed (#575) * fix: submit button state fixed * fix: fixed search in feeds * fix: contact_form_feedback_fixed (#584) * fix: submit button state fixed (#574) * feat: ai chat | bug fixes (#587) * fix: ai chat | design and bugs * feat: ai chat | minor changes * feat: courses | bug fixes (#588) * fix: quiz issues (#589) * feat: courses | bug fixes * feat: courses | bug fixes * feat: courses | bug fixes * feat: community events | Create The UI Of Creating New Event (#591) why: so admin/team member could create new event how: new component , date picker with react day picker * feat: community-events | manage community events add/cancel functionality (#593) * [Feat] Save Event And Validate Inputs How: when user clicks save we check that the values are set properly Why: so the user could create events * [Feat] Manage The Event Actions Depends On Status What: confih different actions depends on event status make the functionality of cancel/restore event actions Why: for every status the actions are different, so the user could cancel or restore event How: config the actions obj * feat: user profile | connections (#594) * feat: encrypted message transfer (#592) * chat message encryption added * feat: encrypted message transfer --------- Co-authored-by: Kabir <[email protected]> * fix: node heap size error * feat: quiz | create options Co-authored-by: Kabir <[email protected]> * fix: readme (#607) * Update IconSpacing (#605) * fix: readme (#608) * fix: readme * fix: security page * feat: Labs and reverseShell fix (#610) * CreateQuiz * Fix ReverseShell * labs --------- Co-authored-by: Kabir <[email protected]> * [Fix] Change From Category to notesCategory (#612) why: to be more specific , in places where there could be more categories how: change From Category to notesCategory * fix: Element Over Menu Header issue (#595) * feat: manage community events | handle edit option (#614) * refactor: Change Status Options why: so there can be only Cancelled or Approved events how: change in relevant places * feat: handle editing event why: so admin could edit event info how: pass this event data to modify-event component * feat: handle editing event why: so admin could edit event info how: pass this event data to modify-event component * fix: package.json & package-lock.json to reduce vulnerabilities (#568) The following vulnerabilities are fixed with an upgrade: - https://snyk.io/vuln/SNYK-JS-AXIOS-6144788 Co-authored-by: snyk-bot <[email protected]> Co-authored-by: ArkadiK94 <[email protected]> * feat: tools | Encoding/decoding (#617) * CreateQuiz * Fix ReverseShell * labs * Signed-off-by: Saedbhati <[email protected]> * Encoder/Decoder --------- Co-authored-by: Arkadi Koifman <[email protected]> * feat: ai chat | prompts * feat: ai prompts are added * feat: ai chat | fix and improve some design and format --------- Co-authored-by: Kabir <[email protected]> * feat: ai chat | api trail feature (#611) * feat: api trial added * feat: ai chat | fix issues --------- Co-authored-by: Kabir <[email protected]> * feat: ai chat | fix issues * feat: community event | connect to redux and server, change Time value types to Date (#623) * [Feat] Manage Events State with Redux and Connect to Server why: to connect the events to backend and managing with redux as other parts in our app how: using redux with axios * [Chore] Change folder name of old events code from Events to OldEvents * [Feat] Pick One or Two Days , Change Time Type why: event can be one day or two days long. better to store startTime and endTime as Date Type how: using different config of dayPicker . write the time in new Date() * [Fix] Change The FilteredEvents With New Type Of Time why: time is type Date so checkings where changed how: see it as a Date type and modify in relevant places * [Fix] Display The Events in Time typeOf Date why: so it will still display proprly how: change in right places * [Chore] disable the "join" button temporarily why: this functionality is not working yet how: comment the lines of code * [Fix] Add Zero At The Beginnig of Value < 10 why: to be compatible with field how: if value < 10 retun `0value` * fix: fix editing date of event bug (#624) why: so admins could edit date of an event how: get in milliseconds and convert to Date stracture * fix: add check for bookmarks data * feat: web security (#625) * feat: web security | base * feat: web security | header and sidebar * feat: websecurity | sidebar fix * feat: websecurity | minor issues fixed * feat: events | event participants, connect to backend and some more fixes (#628) * fix: date input is not valid Date error why: so user could input time and then date how: add check * feat: event participants, connect to backend and some more fixes why: so users could join and leave event what: 1. connect to backend with help of redux, 2. add toast messages which gives more info, 3. show avatars of users how: 1. redux , 2. toastify , 3. populate in backend and use here * feat: CheatSheet/HackBook | Add Responsive Design (#583) * Add Responsive Design to CheatSheet/HackBook Changed the mobile view from 600px to 800px and added toggle features for hamburger menu and made Hack Book responsive for mobile and tablet * Update ProfileElements.jsx removed overflow-x as it was not needed * Add FullScreen Icons * icons shifted to the left and default state of index set to true * feat: profile | auth popup on follow/connect when no user / connection request styling overflow (#622) * auth popup on follow/connect when no user + fixed request overflow * fixed styling for connetion requests and follow btn * minor css typo fix * fixed loading spinners and conncetion btns styling * feat: community events | event details page (#633) * [Feat] Add button to redirect, and redirect to event-details-page why: so user could see more info about event in better way how: react router * [Feat] Show Event Details Page why: so user could see more info for every event in suitable way how: display the info in separate page * [Fix] add a check for case that events array empty why: so the app will not crash * fix: change from icon to "view" button why: looks more clear how: change to button * fix: follow-and-unfollow-functionality (#637) why: so users could follow and unfollow each other how: after the user is follows we need to fetch the followed user data and not our data * feat: quiz | show answers (#609) * Add feature : show answers * feat : skip question button * fix: hover on question buttons * feat: show correct answer * fix : hide skip button * fix : typo * fix : glitch in skipping questions * fix: pass generator (#638) * feat: Feeds Section | Enable Click Functionality for Trending Tags and Comments in Posts (#635) * updated_feeds_section * updated_feeds_section * Update FeedTagsElements.jsx --------- Co-authored-by: unknown <[email protected]> * feat: registration | Add a notification checkbox without functionality (#636) * Added a notification checkbox without functionality * renders with checked checkbox for notifications * fixed spacing between elements * improved positioning of checkbox * Appended 'notifications' to userData object * fixed an error (changed string to boolean) * [Fix] Check if user is not logged in and act accordingly (#643) why: the case that the user is not logged in is not handled so it gives errors, getFeedLikes does not need token how: add checks , getFeedLikes remove token * feat: Bin Exploit (#630) * Bin Exploit * Fix Bug encode to encoder * BinExploit Logo * Change as per request * feat: auth | logout on session expire (#644) * feat: auth | logout on session expire * fix: log out user when session expired why: so when the time finish the user will be logged out how: with help of setTimeout --------- Co-authored-by: ArkadiK94 <[email protected]> * feat: tools | dorking (#648) * style: goals | goal-input-improvement (#647) * feat: Explore | Show Only Users That Connected With Current User (#639) * feat:explore | Show Only Users That Connected With Current User * fix: when open the page display data, when change tab display data. * fix: add check for case: feeds ,blogs, cft can be empty array * fix: show ctf which regiestered by filter (connections/followers/following) why: so when user will see that his followers regiestered to some ctf they will want also to regiester how: check that at least one of the filter user regiestered to ctf * fix: userDetails have null values in array remove them * refactor: change from map() to filter() where needed * fix: update selectedFilter data when user logout --------- Co-authored-by: ArkadiK94 <[email protected]> * feat: blogs | blog filter option is added (#631) * feat: blog filter option is added * fix: some extra lines are removed * fix: margins are adjusted * fix: logged in users will only see filter option * fix: minor changes * refactor: userDetailsFunction is created. * feat: Feeds | Implement Multiple Tag Filtering Functionality (#652) * updated-multi-tag-search * updated-multi-tag-search * updated_feeds * clear icon changed * Update src/components/Feeds/Feeds.jsx Co-authored-by: Arkadi Koifman <[email protected]> * Update src/components/Common/SearchInputBox/SearchInputBox.jsx Co-authored-by: Arkadi Koifman <[email protected]> * Update src/components/Feeds/Feeds.jsx Co-authored-by: Arkadi Koifman <[email protected]> * add comment to "\u200b" --------- Co-authored-by: unknown <[email protected]> Co-authored-by: Arkadi Koifman <[email protected]> * docs: add link in readme to the development website (#656) * docs: add link in readme to the development website * fix: url * Update README.md * feat: events | add events timeline display (#661) * feat: add manage timeline action button why: admins/team members could manage the timeline of event how: add action button * refactor: make TimePickerDisplay separate component why: to use it in other components how: create new component * feat: display the time line list with option to add new items to this list why: so admins could manage the time line of an event how: react * fix cancel-icon and style in search input box (#658) * feat: web security && dashboard (#663) * feat: Chat in Dashboard | Connected Users Display (#659) * only connected users are displayed under users * Update src/components/Chat/Sidebar/Sidebar.jsx Co-authored-by: Arkadi Koifman <[email protected]> * fix: isAccepted is verified for connections --------- Co-authored-by: Arkadi Koifman <[email protected]> * feat: Methodology | create the page (#657) * Methodology * Update src/components/Resources/Methodology/MethodologyMain.jsx Co-authored-by: Arkadi Koifman <[email protected]> * Update src/components/Resources/Methodology/MethodologyMain.jsx Co-authored-by: Arkadi Koifman <[email protected]> * Update src/components/Resources/Methodology/MethodologyMain.jsx Co-authored-by: Arkadi Koifman <[email protected]> * Code Update * cursor :pointer --------- Co-authored-by: Arkadi Koifman <[email protected]> Co-authored-by: Kabir <[email protected]> * refactor: SEO and Accessibility are 100% for main page (#650) * SEO and Accessibility are 100% for main page * file_upload.md added to again --------- Co-authored-by: Kabir <[email protected]> * feat: notifications | Added notification toggle buttons (#660) * Added a notification checkbox without functionality * renders with checked checkbox for notifications * fixed spacing between elements * improved positioning of checkbox * Appended 'notifications' to userData object * fixed an error (changed string to boolean) * Added notifications setting layout * Added the required changes --------- Co-authored-by: Kabir <[email protected]> * feat: Feeds | Add Filters Of Users (#651) * added filter feed by following and button styling * removed filter ByFollowing for non auth users --------- Co-authored-by: Arkadi Koifman <[email protected]> Co-authored-by: amine <[email protected]> * fix:minor-issues (#664) * feat: web security (#665) * feat: common sidebar (#666) * feat: common sidebar * feat: common sidebar * feat: common sidebar * feat: common sidebar * fix: common sidebar (#667) * fix: common sidebar * fix: common sidebar * fix: common sidebar issues (#668) * fix: common sidebar * fix: common sidebar * fix: common sidebar * fix: common sidebar * Fix common sidebar issues (#669) * fix: common sidebar * fix: common sidebar * fix: common sidebar * fix: common sidebar * fix: common sidebar * feat: events | timeline functionality (#672) * feat: modify InputEditor component to have textarea why: so for some inputs, we could use textarea how: use textarea html tag * feat: handle timeline items input change, and validation - check no empty why: so users could insert input how: using structure of {id:{},id:{}} for better lookup * feat: victim help page (#673) * fix: fixed the dropdown issue in methodology. (#674) * fixed dropdown isuue * refactor: removed some useless lines * fix: Relocate-Tag-Filtering-Functionality (#671) * relocate-Tag-Filtering-Functionality * update filter * fix: tools | binary exploit | designs (#675) * feat: events timeline backend connect (#676) * feat:connect to backend by using update route & modify data stracture why: so we could update/create the time line of event how: using already used common functions from ModifyCommunityEvent * feat: add Remove button why: so we could remove the last item from timeline list items how: react * refactor: encapsulate the time related functionality in TimePickerDisplay why: so developers could use it in different parts of the app how: refactor * feat: add default element when click Remove btn for last list item how: react why: so there will be always at least one empty element * refactor: showDate change to dateFieldType in TimePickerDisplay why: to have more flexability * refactor: create MultipleSelect common component why: to use it in different places of the app how: create new component from styles that implements in CTF * fix: show empty data warning first then the other * feat: handle multiple selection on startTime, endTime and topic (#678) why: so we could select the relevant date and topic how: use MultipleSelect component * feat: blogs and feeds data added in analytics (#677) * fix: design (#680) * fix: tools | binary exploit | designs * fix: designs * Update Sidebar.jsx (#681) * Update LabsRoom.jsx (#682) * feat: event details display (#683) * feat: add program input for activity/speaker why: to indicate the activity/speaker of subevent how: add input field * refactor: make UserInfo separate component why: to use in other places * feat: add options to UserInfo to be more suitable as a general component why: to use it in different places as needed * feat: display timeline - base with userInfo why: to display the timeline of event how: create new components for it and use UserInfo to show speaker/activity info * refactor: make DurationDisplay a separate component why: to be able to use in different places * feat: display TimelineItem the title, descr,topic,duration why: so users could see the info of sub events * feat: event details display , more changes (#684) * style: change the width of EventDetails Item and descr of TimelineItem * feat: show ongoing events tab first when there are some ongoing events why: more important so show users the ongoing events first * feat: display the number of users that join the event why: great info to show * fix: in feeds make the avatar to be round why: looks better how: change styles * feat: integrate Add to Calendar Button why: so users could add events to their Calendar how: by using "add-to-calendar-button-react" package * Ctf Ui update (#686) * feat: blogs like feature implemented (#685) * blogs like feature added * blogLike is added in redux store * feat: websecurity | crackme new data (#691) * feat: websecurity | crackme new data (#692) * fix:side bar icons | Websecurity (#693) * fix: contact form | email-validation (#697) * fix email-validation * update --------- Co-authored-by: Hassen ben amor <[email protected]> * Fix: Explore | filter-button and filter-container (#702) * Combine all the empty data fields * fix filter-button and filter-container * Delete src/components/Explore/Explore.jsx * Revert "Delete src/components/Explore/Explore.jsx" This reverts commit 517bfc1d5c5ccd6ebac5fd05df82adafaf738b06. * Revert "Combine all the empty data fields" This reverts commit bb670832a2ff330ecc3e7e27754fb62faf3e05fe. * fix: handle null values --------- Co-authored-by: ArkadiK94 <[email protected]> * Explore | Combine all the empty data fields (#698) * Combine all the empty data fields * add renderNotFoundComponents function * small fix * feat: contact form | Sumbitted succefully mechanism (#700) * fix email-validation * update * add new sumbit button to reset * new toast notification added * newest update * newest updateee * newest updateee --------- Co-authored-by: Hassen ben amor <[email protected]> * Fix: feeds | fead post design in explore section (#701) * fix email-validation * update * add new sumbit button to reset * fix the feed posts * revert commit * Revert "revert commit" This reverts commit e8f668d9e4194628ea1760e8dc4c8594e8918b10. * final revert fix --------- Co-authored-by: Hassen ben amor <[email protected]> * style: contact form | update styles (#704) * update contact form * Update src/components/Homepage/ContactForm/ContactFormElements.jsx --------- Co-authored-by: Arkadi Koifman <[email protected]> * fix-username and date overlapping (#708) * fix * new update * new update (the dot is removed) --------- Co-authored-by: Hassen ben amor <[email protected]> * removed fixed width from ContactFormContainer (#712) * feat: public profile | Icon bug fixed (#694) * Icon bug fixed * style: move the icon to be on left side of the number * style: more changes related to position of icons --------- Co-authored-by: Riyazul555 <[email protected]> Co-authored-by: ArkadiK94 <[email protected]> * feat: Ctf page | make it dynamic page (#705) * Ctf page dynamic * ctf leaderboard * fix: ctf | page designs --------- Co-authored-by: Kabir <[email protected]> * reafactor: import statements changed (#706) * reafactor: changed relative to absolute imports * refactor: change from relaive paths to absolute paths how: with help of eslint plugin "no-relative-import-paths" why: will be easier to know from where the file the we import --------- Co-authored-by: ArkadiK94 <[email protected]> * feat: community events | more adjustments (#713) * fix: general minor issues what: Explore -> check for undefined before using method on array HeroElements -> make the main video in main screen on full screen ContantForm -> make the Select a Reason field, more intuitive * style: change the position of duration inputs why: so it will look better how: event -> instead of date picker footer timeline -> grid * fix: change to yyyy-mm-dd placeholder format - lowercase * feat: Mobile Layout, Dispaly SubMenu (#714) * Add feature: Mobile Layout, Dispaly SubMenu * Fixed: Navbar Responsive Issue * FIxed: Navbar Subtabs Click Issue * feat: analytics | Create Analytics Dashboard Page (#703) * WIP: Create Dashboard Analytics layout * WIP: Create charts for analytics page * feat: Create charts for analytics pages * WIP: Create analytics page * feat: Create analytics dashboard page * fix: uninstall "npm" and "i" packages why: was added accidentally * refactor: Dynamically fetch the posts into popular postcards * refactor: Dynamically fetch postcard data * WIP: Refactor analytics page * WIP: Update changes on analytics feature * refactor: Refactor analytics section code * WIP: Refactor analytics code * WIP: Render inner cards with resp charts for top section dynamically * WIP: Refactor analytics layout code * refactor: Refactor analytics section code * refactor: Refactor analytics section code and clean up --------- Co-authored-by: Daniel Okobia <[email protected]> Co-authored-by: ArkadiK94 <[email protected]> * fix: build error (#718) * fix-build-errors,remove 'use client' (#719) * change the default exports (#720) * Fixing errors (#721) * change the default exports * fix: changing from default exports to fix vite problem, and npm audit fix * fix: to dynamic import (#722) why: Because Chart.js is an ESM library, in CommonJS modules you should use a dynamic import * fix: remove react-horizontal-stacked-bar-chart it did a bug with vite (#723) * fix | Fix Failed Stacked Barchart (#726) * fix: Fix the failed stacked barcharts * fix: Center doughnut chart * style: change doughnut chart size to be more suitable why: now it will look better --------- Co-authored-by: ArkadiK94 <[email protected]> * feat: Explore | Option for Non Selected added. (#729) * user filter changed to all user * fix: map the data before we slice it up why: so it will be more correct in this way we will not loss data --------- Co-authored-by: ArkadiK94 <[email protected]> * feat: web security | design (#734) * docs: add responsive_web template (#741) * docs: add responsive_web template why: to open general issue for users to create new responsive_web related issues * fix: change desc * docs: Replace LinkedIn link In Readme (#748) * Replace LinkedIn link In Readme * Remove Extra Space Lines * docs: Change broken coc url (#750) * Change coc url * Update other.yml coc link * Change bug_report.yml coc url * Change doc_report.yml coc url * Change responsive_web.yml coc url * Change feature_request.yml coc url * change the twitter logo to X logo (#753) * feat: events | small modification events (#736) * feat: sort sub events * fix: update eslint-plugin-no-relative-import-paths package * feat: add filter to events "All,Join,Joined,Full" why: so users could navigate to relavant events how: make SidebarFilterButtons common component * feat: dashboard | add link to the user profile image (#761) * add link to the user profile image * fix missing Link import * using Template literals for consitency * fix: goals | prevent form submit if text is empty (#757) * Prevent form submit if text is empty * Display error when input is empty * Code cleanup * fix: change to toast.warn why: looks more suitable as warning how: toast.warn method --------- Co-authored-by: ArkadiK94 <[email protected]> * feat: search common component | tags (#746) * fixed blogs filter by tag + added tags selection indicator * removed selected tags from search box display * refactored tags logic --------- Co-authored-by: ArkadiK94 <[email protected]> * style: goals | syling goals page (#764) * add link to the user profile image * fix missing Link import * using Template literals for consitency * syling goals page * align input and add button, setting min-height for content --------- Co-authored-by: firassssss <[email protected]> * fix: web security | categories and route highlighting #762 * feat: feeds | delete feeds option and add modal window (#765) * feat: add feed option button display why: when user click this button there will be different options as edit / delete and so on how: add FeedOptions component * feat: add edit/delete options in feed options screen and close functionality why: so we will have those options, about the close so user could close the modal window * refactor: make the modal window to be common component why: to use in different places * feat: add feed delete functionality why: so user could delete feeds * fix: fix load comments with post, and don't send another view when user viewed why: so it is better ux, don't get error how: loading condition , typo * feat: in progress - update comment, delete reply comment * fix: user is undefined when the user is not registered (#768) * fixed example.env file name (#771) in the installation guide it is mentioned to copy the content of .env.example into the .env file however the correct file name for the first is example.env * feat: Master quiz , modal for additional info (#763) * add link to the user profile image * fix missing Link import * using Template literals for consitency * add additional info pop Up window for wrong question * removing unnecessary text * fixing the close button postion * fixing the typo in showAdditionalInfo * info button next to correct answer, tooltip * fixing a typo * fix tooltip text * fixing tooltip styling --------- Co-authored-by: Salem-Bourguiba <[email protected]> * Feature/new designs/fixes (#773) * feat: leaderbaord design | fix websecurty design * feat: chatbox * fix: user check added --------- Co-authored-by: ArkadiK94 <[email protected]> * Feature/new designs/fixes (#774) * feat: leaderbaord design | fix websecurty design * feat: chatbox * fix: user check added * fix: api working check added --------- Co-authored-by: ArkadiK94 <[email protected]> * fix: move the api check after hooks (#776) * [Snyk] Upgrade styled-components from 5.3.11 to 6.1.8 (#725) * feat: upgrade styled-components from 5.3.11 to 6.1.8 Snyk has created this PR to upgrade styled-components from 5.3.11 to 6.1.8. See this package in npm: https://www.npmjs.com/package/styled-components See this project in Snyk: https://app.snyk.io/org/kabir0x23/project/56a100f1-3b8e-4637-93f9-fe3df450cb41?utm_source=github&utm_medium=referral&page=upgrade-pr * fix: add $ symbol before not valid dom props why: so they will not be passed forward * chore: configure stylelintrc , install stylelint-plugin-styled-component plugin why: stylelint-plugin-styled-component plugin help with the break changes of new version * fix: fix all styles why: 1. // comment block the plugin so changed to /* */ 2. so they will steel work event in new styled-component version --------- Co-authored-by: snyk-bot <[email protected]> Co-authored-by: ArkadiK94 <[email protected]> * [Snyk] Security upgrade dompurify from 3.0.5 to 3.0.11 (#737) * fix: package.json to reduce vulnerabilities The following vulnerabilities are fixed with an upgrade: - https://snyk.io/vuln/SNYK-JS-DOMPURIFY-6474511 * fix: change react-helmet to react-helmet-async package --------- Co-authored-by: snyk-bot <[email protected]> Co-authored-by: ArkadiK94 <[email protected]> * fix: styles error, add stylelint (#777) * fix: styles * fix: change width:stretch to width:100% * fix: stylelint issues * chore: disable the named-grid-areas-no-invalid stylelint rule * feat: scrolling function for topics (#785) * fix: quizz button position (#784) * fixed selected question indicator (#780) * fix: upgrade sass from 1.55.0 to 1.75.0 (#778) Snyk has created this PR to upgrade sass from 1.55.0 to 1.75.0. See this package in npm: https://www.npmjs.com/package/sass See this project in Snyk: https://app.snyk.io/org/kabir0x23/project/56a100f1-3b8e-4637-93f9-fe3df450cb41?utm_source=github&utm_medium=referral&page=upgrade-pr Co-authored-by: snyk-bot <[email protected]> * fix: upgrade react-router-dom from 6.16.0 to 6.22.3 (#789) Snyk has created this PR to upgrade react-router-dom from 6.16.0 to 6.22.3. See this package in npm: https://www.npmjs.com/package/react-router-dom See this project in Snyk: https://app.snyk.io/org/kabir0x23/project/56a100f1-3b8e-4637-93f9-fe3df450cb41?utm_source=github&utm_medium=referral&page=upgrade-pr Co-authored-by: snyk-bot <[email protected]> * feat: upgrade @reduxjs/toolkit from 1.9.1 to 2.2.3 (#792) Snyk has created this PR to upgrade @reduxjs/toolkit from 1.9.1 to 2.2.3. See this package in npm: https://www.npmjs.com/package/@reduxjs/toolkit See this project in Snyk: https://app.snyk.io/org/kabir0x23/project/56a100f1-3b8e-4637-93f9-fe3df450cb41?utm_source=github&utm_medium=referral&page=upgrade-pr Co-authored-by: snyk-bot <[email protected]> * feat: chatbox | user close functionality (#794) * chat-close-option * chat-close-option --------- Co-authored-by: Saed bhati <[email protected]> * features/bug-fixes/enhancements (#796) * docs: readme | Docs update readme (#799) * Update README with documentation improvements * Update README with documentation improvements * Snyk upgrade ca2c06e7f53b6e73f049bd0680058bd5 (#798) * feat: upgrade date-fns from 2.30.0 to 3.6.0 Snyk has created this PR to upgrade date-fns from 2.30.0 to 3.6.0. See this package in npm: https://www.npmjs.com/package/date-fns See this project in Snyk: https://app.snyk.io/org/kabir0x23/project/56a100f1-3b8e-4637-93f9-fe3df450cb41?utm_source=github&utm_medium=referral&page=upgrade-pr * feat: update all packages * fix: The CJS build of Vite's Node API is deprecated * fix: install eslint and eslint-plugin-n prev version why: because peer dependency error * fix: authPopup warning --------- Co-authored-by: snyk-bot <[email protected]> * Style: AddCommentForm changed (#767) * AddCommentForm Style changed * fix: revert the deleted "file_upload.md" * Fixed Style * fix: remove width:100% because affect different component --------- Co-authored-by: ArkadiK94 <[email protected]> * fest: analytics | Functionality for analytics page (#782) * Display chart data in json * Adding data files for months and hours * Swapping between the datasets * Change custom hook usage * fix: blogs | user authentication (#807) * Update README with documentation improvements * Update README with documentation improvements * Add comment form component and update related functionality * feat: Search Function in chatbox (#809) * Search Function in chatbox * search box outline removed --------- Co-authored-by: Saed bhati <[email protected]> * enhancements/fixed (#810) * fix: [Responsive-Web] Chat sidebar auto-hiding is not in working in small screen (#811) * Update README with documentation improvements * Update README with documentation improvements * Sidebar automatically hides * feat: websecurity topics data (#816) * bug: font coloring to GoalForm > GoalInput (#815) * Added font coloring to GoalForm > GoalInput * added color to goal submit button * Redesigned input and button on dashboard > goals using styled component * feat: functionality for download button (#814) * Download demy file when clicking on the button * Change file data to fit json files * Change the name of the file * fest: Team Page Base (#783) * Team Page * icon packages * icon changes * feat: Show Password Option #779 (#817) * Feature/Add Show Password Option #779 * Eye Color fix * fix state and first click * feat: copy button to encoded + decoded fields (#819) * Basic working example * Adding common props into CodeContainer * Change decoder into the format of the copy component * feat: add floating like comment and share button in blog (#821) * feat/ench/bug fxies (#822) * fix: Using code container in decode fields as well (#823) * refactor: display.css into styled componenets (#824) * Refactor css styles into styled modules * Extract all components to separate file * Fixed styling issue causing cursor position error in Notes section (#829) * feat: feeds | edit delete func (#826) * feat: change modal structure why: to close it in better way with overlay how: use portal * feat: add delete comment functionality who: dispatch with correct method and url * fix: width of profile image in feeds * feat: adding main feed edit option why: so users could edit feed how: changing the ModifyFeed component * refactor: combine feedsSlice and feedsServer with feedComments * fix: icons problem * fix: make small modifications * feat: modify the rtk to connect to new data structure of feeds, add edit option what: the data structure of feeds changed so modify rtk to fit , change the edit option , and make it work also for comments how: change rtk, made modifications why: to fit new data structure, to make edit work * fix: fix delete functionality * fix: minor issues * feat: Notes App | draggable notes and categories (#717) * draft PR for draggable notes and categories * reducers for notes added * motion and reorder working locally * fix: key warning * update note id working * changed button name * slice created for updating notes on component unmount * some comments are removed * Update package.json * Update package-lock.json --------- Co-authored-by: ArkadiK94 <[email protected]> Co-authored-by: Kabir <[email protected]> * Wrong Facebook community social link (#830) * Refactor css styles into styled modules * Extract all components to separate file * Fix broken link * feat: storybook | Add storybook + some common components (#827) * Refactor css styles into styled modules * Extract all components to separate file * add dependencies * Small change * Add Date Display stories * Add duration display story * MultipleSelect story * Search story * Add side bar * Change folder name * Remove default examples * Removed default assets * remove example files * remove example files * File name * Pr changes * Removed non relevant story * style: fix bg color which was accidentally removed (#835) * fix: Remove unnecessary packages (#836) * Refactor css styles into styled modules * Extract all components to separate file * remove unnecessary packages * fix: multiple select label color (#832) * Refactor css styles into styled modules * Extract all components to separate file * Fix select label wrong color * chore: change label text --------- Co-authored-by: ArkadiK94 <[email protected]> * feat: Feeds | Poll component (#839) * Refactor css styles into styled modules * Extract all components to separate file * pollContainer * Add Poll Answers * Small css change * Add storybook for poll components * name changing * feat: some minor changes & new code review data added (#845) * refactor: image upload hook | create + use in modify feed and create blog (#843) * refactor: create useUploadImages hook and use inside ModifyFeed why: to separate this functionality outside of the components how: create hook and common components related to it * refactor: add submit option in the useUploadImages hook * refactor: change variable name * refactor: use the useUploadImages hook inside CreateBlogV2 component * fix: validate max number of uploaded images * bugs/fixes/features (#846) * Storybook usage contributing file (#833) * Refactor css styles into styled modules * Extract all components to separate file * Add storybook guide * Rename Add storybook guide to CONTRIBUTING.md * Removed example - put docs link * Change running comment * feat: 2fa login (#850) * feat: Login/new/feature (#851) * feat: 2fa login * features/fixes --------- Co-authored-by: Abhay Sahu <[email protected]> Co-authored-by: Arun Chandran <[email protected]> Co-authored-by: Rahul <[email protected]> Co-authored-by: Daniele Viola <[email protected]> Co-authored-by: Kabewh <[email protected]> Co-authored-by: Arkadi Koifman <[email protected]> Co-authored-by: snyk-bot <[email protected]> Co-authored-by: Tandin Tshewang <[email protected]> Co-authored-by: Tushar Daiya <[email protected]> Co-authored-by: Ankit Bhusal <[email protected]> Co-authored-by: Priyanshu Tiwari <[email protected]> Co-authored-by: Bhumika Bachchan <[email protected]> Co-authored-by: Mario Randrianomearisoa <[email protected]> Co-authored-by: Conventional Changelog Action <[email protected]> Co-authored-by: Shagun Dubey <[email protected]> Co-authored-by: Saedbhati <[email protected]> Co-authored-by: ParasAnmol <[email protected]> Co-authored-by: Amdadul Hoq Shabbir <[email protected]> Co-authored-by: Sumanjeet <[email protected]> Co-authored-by: Palash Chitnavis <[email protected]> Co-authored-by: Jigar Borde <[email protected]> Co-authored-by: ArkadiK94 <[email protected]> Co-authored-by: Tirthankar <[email protected]> Co-authored-by: salmen gharsellaoui <[email protected]> Co-authored-by: nassim-ayadi <[email protected]> Co-authored-by: ZizouX0 <[email protected]> Co-authored-by: Som401 <[email protected]> Co-authored-by: unknown <[email protected]> Co-authored-by: Happyashbunny <[email protected]> Co-authored-by: Saif_Magouri <[email protected]> Co-authored-by: sky_de_1991 <[email protected]> Co-authored-by: Ben-aoun-1 <[email protected]> Co-authored-by: amine <[email protected]> Co-authored-by: Ragesh <[email protected]> Co-authored-by: Niraj <[email protected]> Co-authored-by: Hassen ben amor <[email protected]> Co-authored-by: Hassen ben amor <[email protected]> Co-authored-by: AhmedKarray005 <[email protected]> Co-authored-by: Ahmed Sinanović <[email protected]> Co-authored-by: Md Riyazul Islam <[email protected]> Co-authored-by: Riyazul555 <[email protected]> Co-authored-by: K-21 <[email protected]> Co-authored-by: SandwichBite <[email protected]> Co-authored-by: Daniel Okobia <[email protected]> Co-authored-by: Daniel Okobia <[email protected]> Co-authored-by: Tamir Abutbul <[email protected]> Co-authored-by: firassssss <[email protected]> Co-authored-by: Adamss10 <[email protected]> Co-authored-by: Salem-Bourguiba <[email protected]> Co-authored-by: firassssss <[email protected]> Co-authored-by: seifnamissi <[email protected]> Co-authored-by: Salem-Bourguiba <[email protected]> Co-authored-by: YoussefMlayah <[email protected]> Co-authored-by: Saed bhati <[email protected]> Co-authored-by: Krishna Pathak <[email protected]> Co-authored-by: Alvaro Torres <[email protected]> Co-authored-by: Prathamesh Kothalkar <[email protected]>
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.
Fixes Issue
Closes #459
Changes proposed
Screenshots
20231004193118.mp4
Note to reviewers
I checked the loading property in the state of different modals from the redux, while it is true the spinner will appear. I didn't create new state , so I didn't manage state instead I used already existing one
Code of Conduct
Check List (Check all the applicable boxes)
You can also join our Discord community.
Feel free to check out other cool repositories of the Thecyberworld.
Join the Thecyberworld GitHub Organisation by raising an issue (you will be sent an invitation).