Skip to content

Development to Master (version update v0.0.4 to v0.0.5)#234

Merged
leekahung merged 339 commits intoMasterfrom
Development
Jun 9, 2023
Merged

Development to Master (version update v0.0.4 to v0.0.5)#234
leekahung merged 339 commits intoMasterfrom
Development

Conversation

@leekahung
Copy link
Contributor

@leekahung leekahung commented Jun 7, 2023

This merge will incorporate the following changes listed from CHANGELOG.md as v0.0.4 in Master. It's been a while since our last update with version 0.0.3 (April 25th, 2023), but lots of new features been added. See below.

Note to Developers: In case you're encountering errors with regards to userlist.ttl such as not being able to delete users or see the person's webId, it might mean that you're still using an older version of userlist.ttl. In this update, userlist.ttl has been revamped to handle new input fields in a slightly different manner. To reset the userlist.ttl, simply delete it from your Solid Pod under Users, log out of PASS and log back in. A new userlist.ttl will be generated for you in the Pod and you'll be able to add/delete users again.

Visual Changes Since v0.0.4

Home

Screen Shot 2023-06-07 at 1 43 07 (2)

Clients (formerly Home)

Screen Shot 2023-06-07 at 1 43 17 (2)

Screen Shot 2023-06-07 at 1 43 24 (2)

Screen Shot 2023-06-07 at 1 43 28 (2)

Forms

Screen Shot 2023-06-08 at 14 34 59 (2)

Inbox

Screen Shot 2023-06-07 at 1 43 37 (2)

Screen Shot 2023-06-07 at 1 47 54 (2)

Outbox

Screen Shot 2023-06-07 at 1 43 40 (2)

CHANGELOG - v0.0.5 (June 7, 2023)

Features

Fixes

Dev Changes

Others

leekahung and others added 30 commits May 24, 2023 00:58
Merging PR into Development. This PR reorganizes the utils directory into 4 subdirectories.
Merging fixes to Footer margin into Development.
…es; Refactored rendering of user list with && instead of ternary
… generateType for create or update and webId parameters; Updated both set permission forms to pass Access objects directly; Corrected working of cross-pod uploads; Updated all functions that uses createDocAclForUser to use setDocAclForUser parameters
src/theme.js Outdated
typography: {
fontFamily: 'Roboto, sans-serif'
fontFamily: 'Roboto, sans-serif',
dd: { fontWeight: 'bold' },
Copy link
Contributor

Choose a reason for hiding this comment

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

Ok, so there is a "dd" in the theme - man, I'm sorry. Jumped the gun on some comments in other files.

BUT, the font doesn't actually display as bold - so this is probably not implemented correctly.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thinking of replacing it with just sx locally, think it'll be better than setting custom variants since the main point of the status message is just to indicate to the user what's happening.

<div style={formRowStyle}>
<label htmlFor="upload-doc">Select document type to upload: </label>
<label htmlFor="upload-doc">Select Document Type</label>
<DocumentSelection htmlId="upload-doc" />
Copy link
Contributor Author

Choose a reason for hiding this comment

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

You're correct, but the htmlId here is used as a prop to pass in the value for id, see <select> in DocumentSelection.jsx where htmlId used as a prop and used as the id attribute

@xscottxbrownx
Copy link
Contributor

@timbot1789 if you have time to look over this. It's 105 changed files.

You are more than welcome to look over all of them, but I think it would be really helpful to split up. I think for the sake of the project you would be better to review the utils, tests, and contexts files. I'm almost done going over the rest (at 73 of 105 reviewed.)

@leekahung
Copy link
Contributor Author

leekahung commented Jun 8, 2023

Alright, gonna attempt some refactors for one of the issues mentioned in the comments here and put it in another PR to be merged to Development. It's pretty sizable so I've pushed it all there #236. If it looks good @xscottxbrownx, we'll merge it into this.

leekahung added 2 commits June 8, 2023 12:25
…lection

Merging PR into Development. This PR include fixes to Development to Master and also refactors DocumentSelection.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Really minor, but have noticed local pod URL was misconfigured after Adam mentioned it on Discord.

Even though you could still log into PASS just fine with http://localhost:3000 and it will correct the right webId, it will generate a webId without the / at the end of the Solid Server (notice localhost:3000profile without the slash). Even though the final webId would still need to be altered to http://localhost:3000/test/profile/card#me, could easily fix the localhost:3000profile problem by including / to VITE_SOLID_IDENTITY_PROVIDER_DEV.

Screen Shot 2023-06-08 at 22 31 56

Copy link
Contributor

Choose a reason for hiding this comment

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

looks like you found and fixed

Copy link
Contributor

@xscottxbrownx xscottxbrownx left a comment

Choose a reason for hiding this comment

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

Approving all files that I reviewed.

I did not review:

  • contexts folder
  • hooks folder
  • model-helpers folder
  • utils folder
  • test folder

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

version upgrade Updating Master from Development

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants