Skip to content

use "stream" instead of the deprecated "append" - #303

Closed
Mecit wants to merge 2 commits into
dwyl:mainfrom
Mecit:patch-1
Closed

use "stream" instead of the deprecated "append"#303
Mecit wants to merge 2 commits into
dwyl:mainfrom
Mecit:patch-1

Conversation

@Mecit

@Mecit Mecit commented Aug 4, 2025

Copy link
Copy Markdown

No description provided.

nelsonic
nelsonic previously approved these changes Aug 5, 2025

@nelsonic nelsonic left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This is a great update. Thanks. 🙏
Just need to figure out why the build doesn't pass ... 💭

@Mecit

Mecit commented Aug 6, 2025

Copy link
Copy Markdown
Author

Hey, @nelsonic. I forgot to update the attributes for the view in my first commit so I sent another including the changes for that. Sorry about that. As for the build, I can see some tests are failing in auth_controller_test.exs. I haven't run the project locally, so I have no idea why that's happening. Could there be a problem with generating JWT tokens?

@Mecit Mecit closed this by deleting the head repository Nov 5, 2025
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.

2 participants