Next page button in users/assignment view don't take into account search criteria

llaske / sugarizer-server

Sugarizer Server allow deployment of Sugarizer on a local server, for example on a school server.

Apache License 2.0

43 stars 94 forks source link

Next page button in users/assignment view don't take into account search criteria #364

Open llaske opened 1 year ago

llaske commented 1 year ago

In the Users view and in the Assignment view, the next button don't take into account search criteria.

See the problem below (for the Users view):

cNyOEOR9W8

arian0zen commented 1 year ago

hey @llaske , was asking that I want to work on this issue.. but I am not familiar on how to set up the developer enviornment for this sugarizer server, can you help me a bit with this ? or am I missing something obvious here ?

update: 10:38 pm 25th feb oh I am sorry, I was confused where is the repo for this sugarizer and I was looking in the main sugarlabs repo... so do I clone this repo llaske/sugarizer-server and set up the enviornment ?

another update query: 11:17 pm 25th feb I was carefully looking at the cource code on different folders from both sugarizer and sugarizer server.. I felt i am familiar with the .js codes and also I am confident I can work on a few of features raised in the issues.. but, Unfortunately I am not at all familiar with the docker and staff so I am finding it a bit difficult set up to start to code.. I really really need help on thisss. hey @NikhilM98 I am mentioning you here as well, if you can help me with an way around, thankkk you, looking forward :)

llaske commented 1 year ago

@arian0zen just follow running instructions here and you will have a running environment

arian0zen commented 1 year ago

do I have keep to both sugarizer and sugarizer-server folder in the same directory ? what should be the heierarchy ?

llaske commented 1 year ago

Not in the same directory, side by side. Clone commands create a sugarizer and a sugarizer-server directory at the same level.

arian0zen commented 1 year ago

hey, I kinda got the set up ready, though sh generate-docker-compose.sh this command is not working for me, I am getting this error, (i am on windows 10)

why is this ? though when I am using nodemon sugarizer.js I see the local host working on 8080.

So, I was asking should I stop worrying about why docker compose is not working in my setup and start working on the features ? or sh generate-docker-compose.sh this command is necessary ?

do I have to install docker first like this ? is it necessary for development ?

llaske commented 1 year ago

@arian0zen it look like you're not familiar with Linux and Shell command line. It will be difficult for you to contribute without a basic understanding of it. I suggest you to first increase your skill on Linux before trying to play with docker and before starting your contributions.

arian0zen commented 1 year ago

Okkkkaay understoood

parteekcoder commented 1 year ago

hey, I kinda got the set up ready, though sh generate-docker-compose.sh this command is not working for me, I am getting this error, (i am on windows 10)

why is this ? though when I am using nodemon sugarizer.js I see the local host working on 8080.

So, I was asking should I stop worrying about why docker compose is not working in my setup and start working on the features ? or sh generate-docker-compose.sh this command is necessary ?

do I have to install docker first like this ? is it necessary for development ?

sh command will not work on windows

llaske commented 1 year ago

Fixed in https://github.com/llaske/sugarizer-server/pull/407