Skip to content

Commit e9dc796

Browse files
authored
Merge pull request open-source-labs#243 from open-source-labs/rose/fixes
Fixed: Final ReacType v19
2 parents 100e825 + 623efbe commit e9dc796

File tree

6 files changed

+38
-23
lines changed

6 files changed

+38
-23
lines changed

CHANGE_LOG.md

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -10,15 +10,15 @@ Changes:<br>
1010
- Developer Improvement:
1111
- Typescript conversion continued and now sits at ~95%
1212
- User Features:
13-
- Collaboration Room:
13+
- Collaboration Room:
1414
- Bug Fixes:
1515
- Debug “Leave Room” functionality removing username from the users list
1616
- Debug “Join Room” functionality so the current canvas does not reset upon new user joining collaboration
1717
- Debug Code Preview button that sent error if toggled more than once and does not force toggled view to other users in the room
1818
- Collaboration room is now ready for release (v1)
1919
- Implemented live cursor tracking with on/off function where multiple users can choose to see the other users mouse cursors in real-time in the same canvas.
2020
- Added keydown functionality to “Join Room” by triggering button click on keyboard “Enter”
21-
- Reconfigured web socket initiation to force new connection when joining room
21+
- Reconfigured web socket initiation to force new connection when joining room
2222
- Refactored the way changes were being passed to other users in the collaboration room
2323
- Significantly reduces the amount of data being passed among users by passing only the payload for each individual action, triggering singular updates for other users in the collaboration environment
2424
- Added Event Emitters for each action that updates canvas
@@ -33,7 +33,7 @@ Changes:<br>
3333

3434
Recommendations for Future Enhancements:<br>
3535

36-
- Fix Undo & Redo functionality. Undo & Redo buttons on the customization page not functioning as expected.
36+
- Fix Undo & Redo functionality. Undo & Redo buttons on the customization page not functioning as expected.
3737
- Update Electron for desktop application use. Resolve electron app functionality to coincide with web app functionality.
3838
- Add Change Log/View Edit History feature in app
3939
- v.17 recommendations regarding the Marketplace are still undeveloped.
@@ -76,20 +76,20 @@ Changes:<br>
7676
- Join/Nickname Button:
7777
- Allows users to specify which room to join, and what name to display upon joining the room.
7878
- The button only shows when the user is not connected to the room, requiring both fields to be filled out.
79-
79+
8080
Recommendations for Future Enhancements:<br>
8181

8282
- Chat functionality so users in the same room can discuss their projects.
83-
- List of active rooms so users can simply pick one to join. Will likely be paired with a password feature for security, so only users with the proper credentials can join a particular room.
83+
- List of active rooms so users can simply pick one to join. Will likely be paired with a password feature for security, so only users with the proper credentials can join a particular room.
8484
- True real-time rendering so users can see components as they're being dragged onto the canvas, rather than only when they're placed.
85-
- Optimize performance of room state updates
85+
- Optimize performance of room state updates
8686
- v.17 recommendations regarding the Marketplace are still undeveloped.
8787
- Solve residual bugs. Undo & Redo buttons on the customization page not functioning as expected. Backend bugs persist as seen in the console when running the dev environment. Persistent Redux error that causes page to rerender more often than necessary.
88-
- Resolve electron app functionality to coincide with web app functionality.
88+
- Resolve electron app functionality to coincide with web app functionality.
8989
- For the state manager option in the data table there is a MuiData-menu that is not visible when clicking it and after the filter option is clicked it creates a white space in the bottom of the page.
90-
- Expand testing coverage. Continue fixing old tests which rely on outdated dependencies, and implementing new tests.
91-
- Continue modularizing code. Many large, unwieldy files that should be broken up into more modular components still exist.
92-
- Continue Typescript conversion. Consider toggling noImplicitAny to find all 'any' cases that can be addressed.
90+
- Expand testing coverage. Continue fixing old tests which rely on outdated dependencies, and implementing new tests.
91+
- Continue modularizing code. Many large, unwieldy files that should be broken up into more modular components still exist.
92+
- Continue Typescript conversion. Consider toggling noImplicitAny to find all 'any' cases that can be addressed.
9393
- Continue cleanup of outdated and unused code and files.
9494
- Collaboration feature still needs to be improved for scalability with AWS servers and clients for better experiences. The feature currently is limited to access with only 1 AWS cloud server.
9595
- Future teams could look into data structures for scaling on the server side of the app to improve data transmitting and multiple server functionality.
@@ -112,15 +112,15 @@ Changes:<br>
112112
- UI updated with a modern style for a better developer experience
113113
- Added many user feedback alerts for a better experience including alerts for when projects are published, cloned, deleted, HTML custom tags are created, context created, or custom components created.
114114
- Built a specific buttons menu that individually displays the HTML elements, reusable components created, and join room option.
115-
- Redesigned the state manager panel option to be readable and functional.
115+
- Redesigned the state manager panel option to be readable and functional.
116116
- Drop down menu now closes only when the user clicks outside of the menu
117117
- Marketplace:
118118
- Implemented a dedicated area for developers to share their projects
119119
- Routing handled by React Router
120120
- Projects can also be cloned to the user's account to be used and edited with the addition of a button
121121
- Added search functionality to search by username and project name
122122
- Included a separate section in the Saved Projects and Delete Projects modal in the Manage Project menu for cloned projects from the Marketplace
123-
- Publish/Unpublish Button:
123+
- Publish/Unpublish Button:
124124
- Publish feature on the web app allows users to publish their saved project files into the Marketplace from the main app page
125125
- Dynamically switches between publish/unpublish depending on whether the loaded project is in the Marketplace
126126

Dockerrun.aws.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212
"Environment": [
1313
{
1414
"Name": "API_BASE_URL",
15-
"Value": "Reactype-v17env.eba-sw2fhsbj.us-east-1.elasticbeanstalk.com"
15+
"Value": "Reactype-v19env.eba-sw2fhsbj.us-east-1.elasticbeanstalk.com"
1616
}
1717
]
1818
}

app/src/components/left/RoomsContainer.tsx

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -333,12 +333,15 @@ const RoomsContainer = () => {
333333
<Typography variant="h5" color={'#f2fbf8'}>
334334
Live Room: {roomCode}
335335
</Typography>
336-
<Typography variant="h6" color={userColors[userList.indexOf(userName)]}>
337-
Nickname: {userName}
338-
</Typography>
339336
{/* Set up condition rendering depends on if user joined a room then render leave button if not render join button */}
340337
{userJoined ? (
341338
<>
339+
<Typography
340+
variant="h6"
341+
color={userColors[userList.indexOf(userName)]}
342+
>
343+
Nickname: {userName}
344+
</Typography>
342345
<Typography
343346
variant="body1"
344347
sx={{

app/src/components/main/Canvas.tsx

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,6 @@ import { combineStyles } from '../../helperFunctions/combineStyles';
1616
import renderChildren from '../../helperFunctions/renderChildren';
1717
import { emitEvent, getSocket } from '../../helperFunctions/socket';
1818
import { FaMousePointer } from 'react-icons/fa';
19-
import { Console } from 'console';
2019

2120
function Canvas(props: {}): JSX.Element {
2221
const state = useSelector((store: RootState) => store.appState);
@@ -97,10 +96,10 @@ function Canvas(props: {}): JSX.Element {
9796
// Removes the mouse cursor of the user that leaves the collaboration room.
9897
const handleCursorDeleteFromServer = () => {
9998
setRemoteCursors((prevRemoteCursors) =>
100-
prevRemoteCursors.map((cursor) => ({
101-
...cursor,
102-
isVisible: userList.includes(cursor.remoteUserName)
103-
}))
99+
// filter cursors to include only those in the userList
100+
prevRemoteCursors.filter((cursor) =>
101+
userList.includes(cursor.remoteUserName)
102+
)
104103
);
105104
};
106105

app/src/public/index-prod.ejs

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,12 +2,19 @@
22
<html lang="en-US">
33
<head>
44
<meta charset="UTF-8" />
5-
<script src="https://kit.fontawesome.com/7e1cebd082.js" crossorigin="anonymous"></script>
5+
<script
6+
src="https://kit.fontawesome.com/7e1cebd082.js"
7+
crossorigin="anonymous"
8+
></script>
69
<meta
710
property="csp-nonce"
811
content="<%= htmlWebpackPlugin.options.nonce %>"
912
/>
1013
<title>ReacType</title>
14+
<link
15+
rel="icon"
16+
href="https://raw.githubusercontent.com/open-source-labs/ReacType/master/app/src/public/icons/png/512x512.png"
17+
/>
1118
</head>
1219
<body>
1320
<div id="app"></div>

app/src/redux/reducers/slice/appStateSlice.ts

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -778,14 +778,20 @@ const appStateSlice = createSlice({
778778
style: {}
779779
};
780780
const components = [rootComponent];
781+
782+
const stylesheet = '';
783+
const resetHTMLTypes = HTMLTypes;
784+
781785
return {
782786
...state,
783787
nextChildId,
784788
nextTopSeparatorId,
785789
rootComponents,
786790
nextComponentId,
787791
components,
788-
canvasFocus
792+
canvasFocus,
793+
stylesheet,
794+
HTMLTypes: resetHTMLTypes
789795
};
790796
},
791797
updateProjectName: (state, action) => {

0 commit comments

Comments
 (0)